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 handling UI #33

Open
srallen opened this issue Feb 21, 2017 · 3 comments
Open

Error handling UI #33

srallen opened this issue Feb 21, 2017 · 3 comments
Milestone

Comments

@srallen
Copy link
Contributor

srallen commented Feb 21, 2017

No description provided.

@amy-langley amy-langley added this to the future milestone Feb 23, 2017
@mcbouslog
Copy link
Contributor

is this initially adding appropriate notifications throughout the app, similar to the collaborator error catch?

@rogerhutchings
Copy link
Contributor

Side note - this may be an opportunity to try addressing zooniverse/rfc#20 with Honeybadger.

@srallen
Copy link
Contributor Author

srallen commented Jun 13, 2017

There is a UI now, thanks to grommet, but I went through and generically added a handler to almost much all the promise catches. There may be some instances where we don't need to report all the errors (like when a 404 is expected) or cases where the app notification should appear when we need to inform the user of something (such as a warning or success state for a change maybe?).

It'd be a good idea to add Honeybadger too, but that's for us, not for the user.

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

4 participants