Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error Messages are Vague (and possibly other responses) #2

Open
Hybridx24 opened this issue Dec 17, 2019 · 1 comment
Open

Error Messages are Vague (and possibly other responses) #2

Hybridx24 opened this issue Dec 17, 2019 · 1 comment

Comments

@Hybridx24
Copy link

Hybridx24 commented Dec 17, 2019

Hi!

I wanted to start off by saying this is an awesome project!

Having used it, I noticed that whenever I get an error with table.record(), I get a vague error message saying "4xx add error". This makes it very hard to diagnose any potential issues and would be much more effective if it included some context/cause for the error (which is provided by the Airtable response).

I was able to add the error information to the console error post by modifying line 100 in airpuck.js from:
console.log(xhr.status) to console.log(xhr.status, JSON.parse(xhr.responseText)). I imagine this could be done for the other request types as well (UPDATE, DELETE, etc).

@justkd
Copy link
Owner

justkd commented Jan 2, 2020

Hey sorry for the delay in responding. Thanks for the feedback. I actually have been adding more detailed logging to other projects. I'll make it a point to update this one in similar fashion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants