Skip to content

Conversation

@jennmueng
Copy link
Member

@jennmueng jennmueng commented Oct 30, 2025

Remember a user's last used root cause card action in the user options

CleanShot 2025-10-30 at 21 18 27@2x CleanShot 2025-10-30 at 21 16 51@2x

Will still show [Find Solution] if you don't have any integrations configured.

Paired with #102400

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 30, 2025
@jennmueng jennmueng marked this pull request as ready for review October 30, 2025 15:33
@jennmueng jennmueng requested a review from a team as a code owner October 30, 2025 15:33
@jennmueng jennmueng requested review from a team and roaga October 30, 2025 15:33
Copy link
Member

@roaga roaga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be a lot simpler to just use local storage. See the autofixStartBox and the "Start Root Cause Analysis" button, which does the same thing you're trying to do for this button

@jennmueng
Copy link
Member Author

It might be a lot simpler to just use local storage. See the autofixStartBox and the "Start Root Cause Analysis" button, which does the same thing you're trying to do for this button

Yep thats what I was originally thinking of doing but realized that hmm, a user setting actually works here and persists across devices (for example, I use sentry on my personal laptop too)

@roaga
Copy link
Member

roaga commented Oct 30, 2025

It might be a lot simpler to just use local storage. See the autofixStartBox and the "Start Root Cause Analysis" button, which does the same thing you're trying to do for this button

Yep thats what I was originally thinking of doing but realized that hmm, a user setting actually works here and persists across devices (for example, I use sentry on my personal laptop too)

Idk I think that's over-engineering it for something that's just a small UI convenience. You wouldn't need backend work for it either

@jennmueng
Copy link
Member Author

It might be a lot simpler to just use local storage. See the autofixStartBox and the "Start Root Cause Analysis" button, which does the same thing you're trying to do for this button

Yep thats what I was originally thinking of doing but realized that hmm, a user setting actually works here and persists across devices (for example, I use sentry on my personal laptop too)

Idk I think that's over-engineering it for something that's just a small UI convenience. You wouldn't need backend work for it either

I agree, but for the sake of this work the backend work is already done #102400, i'll just undo it for the sake of getting this in and I don't want waste too much time on a convo on this

@jennmueng jennmueng merged commit 5a30991 into master Oct 31, 2025
47 checks passed
@jennmueng jennmueng deleted the feat/seer-last-used-solution-fe branch October 31, 2025 16:40
jjbayer pushed a commit that referenced this pull request Nov 3, 2025
Remember a user's last used root cause card action in the user options

<img width="914" height="322" alt="CleanShot 2025-10-30 at 21 18 27@2x"
src="https://github.com/user-attachments/assets/bcfe843a-f707-4f39-a3eb-e020e66e66e4"
/>
<img width="772" height="360" alt="CleanShot 2025-10-30 at 21 16 51@2x"
src="https://github.com/user-attachments/assets/862572cc-86c8-4228-9901-a341e750c138"
/>

Will still show [Find Solution] if you don't have any integrations
configured.

Paired with #102400
shashjar pushed a commit that referenced this pull request Nov 4, 2025
Remember a user's last used root cause card action in the user options

<img width="914" height="322" alt="CleanShot 2025-10-30 at 21 18 27@2x"
src="https://github.com/user-attachments/assets/bcfe843a-f707-4f39-a3eb-e020e66e66e4"
/>
<img width="772" height="360" alt="CleanShot 2025-10-30 at 21 16 51@2x"
src="https://github.com/user-attachments/assets/862572cc-86c8-4228-9901-a341e750c138"
/>

Will still show [Find Solution] if you don't have any integrations
configured.

Paired with #102400
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
Remember a user's last used root cause card action in the user options

<img width="914" height="322" alt="CleanShot 2025-10-30 at 21 18 27@2x"
src="https://github.com/user-attachments/assets/bcfe843a-f707-4f39-a3eb-e020e66e66e4"
/>
<img width="772" height="360" alt="CleanShot 2025-10-30 at 21 16 51@2x"
src="https://github.com/user-attachments/assets/862572cc-86c8-4228-9901-a341e750c138"
/>

Will still show [Find Solution] if you don't have any integrations
configured.

Paired with #102400
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants