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

[APM] Error overview has local filter for transaction.result #45264

Closed
sorenlouv opened this issue Sep 10, 2019 · 3 comments · Fixed by #45379
Closed

[APM] Error overview has local filter for transaction.result #45264

sorenlouv opened this issue Sep 10, 2019 · 3 comments · Fixed by #45379
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support

Comments

@sorenlouv
Copy link
Member

On the errors overview page the local filter allows the user to filter by transaction.result. This doens't make much sense since errors never have this field.

This causes issues when linking from transactions to errors if a transaction.result is selected, since this is carried over and causes the error to not be found.

Screen Shot 2019-09-10 at 16.15.26.png

@sorenlouv sorenlouv added [zube]: Inbox bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support labels Sep 10, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui

@dgieselaar
Copy link
Member

Just to be sure, this is mentioned in the design doc: https://docs.google.com/document/d/1xHkMamVZA-thBY2FOWGZOLkeId_0eJkZ3vfRqGOfRPY/edit. I guess that was a mistake then?

@sorenlouv
Copy link
Member Author

I guess that was a mistake then?

I think so. I don't understand when this would be relevant since errors don't have that field.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants