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
feat(new-trace): Added logic for red errored rows. #66194
Conversation
Abdkhan14
commented
Mar 4, 2024
@@ -1219,6 +1221,24 @@ export class ParentAutogroupNode extends TraceTreeNode<TraceTree.ChildrenAutogro | |||
} | |||
return this.tail.children; | |||
} | |||
|
|||
get errored(): boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
has_error since this property is static, we should cache it avoid iterating the entire tree. I would probably go with having a TraceTreeNode[] property on the autogrouped node where we push the errored references at the time of autogrouping. This also gives us the ability to show to the user all errors or use that list to iterate and show 🔥 icons.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea @JonasBa , I'll move forward with this.
<img width="1035" alt="Screenshot 2024-03-03 at 9 36 22 PM" src="https://github.com/getsentry/sentry/assets/60121741/860a6370-b1aa-41dc-b5c8-b62b9778ffa5"> --------- Co-authored-by: Abdullah Khan <abdullahkhan@PG9Y57YDXQ.local>