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

[Tracking] Add error handling #1

Open
3 tasks
wschella opened this issue May 15, 2021 · 0 comments
Open
3 tasks

[Tracking] Add error handling #1

wschella opened this issue May 15, 2021 · 0 comments

Comments

@wschella
Copy link
Collaborator

To make a robust and deploy-ready app, the current code needs at least some error handling, I think providing handlers for following logic provide the most value:

  • mu-search server calls. The server can be down or slow, and we could have introduced a bug w.r.t. query parameters. The application can't quite continue in this case, as there are no search results to render.
  • Rendering a (single) search result. In most cases an error in rendering a search result would also mean errors in the other search results, but there are plenty of possibilities it's because of an unexpected attribute value in a single result (e.g. list where a single value was expected, a missing field, ...), and this way you could still display partial results.
  • Catch-all application error. We should at least display something nice & a link to the issue list. This would also catch errors to mu-resource.
@wschella wschella changed the title [Tracking] Error handling [Tracking] Add error handling May 15, 2021
@wschella wschella mentioned this issue May 15, 2021
8 tasks
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

1 participant