-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add Error Handling in the frontend #18
Comments
can I work on this issue? |
Yes you can go ahead, would recommend you to use React Toastify to show the errors and also success messages if any. don't show success messages like data got loaded etc, only if there's error fetching data you can say something like Something went wrong, etc. So what we need to check is also if the status is ok or failed, (even after we get a successful response ) and if the status is failed add a error message where the message can be simply the same that we get from the response as shown in the above image |
I would assign this to you, just please give me a date for your 1st PR submission, it doesn't need to complete implementation, just a small part to track your progress and that would be good. |
16 Dec 2022. |
So right now whenever we make an API request , and if we might face an error the only thing done is that we console.log the error , which is bad with respect to user perspective. I would suggest to have a npm package and to show errors properly if anything goes wrong
The text was updated successfully, but these errors were encountered: