Skip to content

feat(replay): add hovercard to release tag and filter dropdown #90701

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

Merged
merged 1 commit into from
May 5, 2025

Conversation

michellewzhang
Copy link
Member

@michellewzhang michellewzhang commented Apr 30, 2025

closes #90633

  • adds a release hovercard to the release tag in replay details
  • previous behavior was that if the tag was clicked, it would populate the replay search bar with the release:xxx filter
  • new behavior is a dropdown that has 2 options: same search behavior as before, or going to release details
  • also removes deprecated browserHistory from the replay index table cell dropdown filters
Screen.Recording.2025-04-30.at.1.39.33.PM.mov

@michellewzhang michellewzhang requested a review from a team as a code owner April 30, 2025 20:46
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 30, 2025
@michellewzhang michellewzhang changed the title ref(replay): add hovercard to release tag and filter dropdown feat(replay): add hovercard to release tag and filter dropdown May 1, 2025
@billyvg
Copy link
Member

billyvg commented May 1, 2025

@michellewzhang just to clarify, what happens when you click on the release version now, is it still linking to search w/ release?

@michellewzhang
Copy link
Member Author

@michellewzhang just to clarify, what happens when you click on the release version now, is it still linking to search w/ release?

nothing happens when you click -- you have to click the 3 dots to see the 2 options

@@ -25,6 +29,8 @@ const expandedViewKeys = [
'sdk.replay.maskingRules',
];

const releaseKeys = ['release', 'releases'];
Copy link
Member

Choose a reason for hiding this comment

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

l: this could be a Set

label: t('Go to release details'),
onAction: () =>
navigate(
makeReleasesPathname({
Copy link
Member

Choose a reason for hiding this comment

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

Note for myself, I'll have to update these calls when #90428 lands

@michellewzhang michellewzhang merged commit 3122883 into master May 5, 2025
45 checks passed
@michellewzhang michellewzhang deleted the mz/replay-release-hover branch May 5, 2025 15:54
Copy link

codecov bot commented May 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #90701      +/-   ##
==========================================
- Coverage   87.84%   84.25%   -3.60%     
==========================================
  Files       10265    10265              
  Lines      580837   580835       -2     
  Branches    22569    22568       -1     
==========================================
- Hits       510235   489366   -20869     
- Misses      70169    91036   +20867     
  Partials      433      433              

andrewshie-sentry pushed a commit that referenced this pull request May 12, 2025
closes #90633

- adds a release hovercard to the `release` tag in replay details
- previous behavior was that if the tag was clicked, it would populate
the replay search bar with the `release:xxx` filter
- new behavior is a dropdown that has 2 options: same search behavior as
before, or going to release details
- also removes deprecated `browserHistory` from the replay index table
cell dropdown filters


https://github.com/user-attachments/assets/6d5a0fa8-aea2-4c72-989d-84a31ba8e05b
@github-actions github-actions bot locked and limited conversation to collaborators May 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Replay] Add release context on hover, and link to fly out
2 participants