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.

Sign up for GitHub

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

Jump to bottom

[APM] Display errors to user instead of swallowing them #40986

Closed
sorenlouv opened this issue Jul 12, 2019 · 2 comments · Fixed by #42160
Closed

[APM] Display errors to user instead of swallowing them #40986

sorenlouv opened this issue Jul 12, 2019 · 2 comments · Fixed by #42160
Assignees
@sorenlouv
Labels
enhancement New value added to drive a business result Team:APM All issues that need APM UI Team support

Comments

@sorenlouv
Copy link
Member

APM UI doesn't handle errors very well. They are mostly caught but not handled or displayed to the user.

Example: Service list page. If an error occurs while fetching the list of services the user won't know. Instead they are told that there are no services.

Screen Shot 2019-07-12 at 14.16.56.png
@sorenlouv sorenlouv added [zube]: Inbox Team:APM All issues that need APM UI Team support labels Jul 12, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui

@sorenlouv sorenlouv added the enhancement New value added to drive a business result label Jul 12, 2019
@sorenlouv sorenlouv mentioned this issue Jul 17, 2019
Closed
@sorenlouv
Copy link
Member Author

First phase: just show the error from the backend
Second phase: show more user-friendly errors.

@sorenlouv sorenlouv self-assigned this Jul 29, 2019
@sorenlouv sorenlouv mentioned this issue Jul 29, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sorenlouv sorenlouv

Labels
enhancement New value added to drive a business result Team:APM All issues that need APM UI Team support
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[APM] Surface http errors to users sorenlouv/kibana
2 participants
@sorenlouv @elasticmachine