Skip to content

Conversation

@jerryzhou196
Copy link
Contributor

@jerryzhou196 jerryzhou196 commented Dec 2, 2025

Related to REPLAY-814: Implement new UI for Replay Page

Change columns to support generic link objects. This is used in #104246 to maintain the user email search feature.

…ation

Modified ReplayPreviewPlayer and ReplayTable components to utilize linkQuery for constructing navigation paths. This change enhances the clarity of the code and ensures consistent handling of query parameters across replay-related components.
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 2, 2025
@linear
Copy link

linear bot commented Dec 2, 2025

@jerryzhou196
Copy link
Contributor Author

Tested links for:

  • Replay Table in Issue Detail
  • Replay Table in Insights
  • Replay Table in Replay Index

@jerryzhou196 jerryzhou196 marked this pull request as ready for review December 2, 2025 16:55
@jerryzhou196 jerryzhou196 requested a review from a team as a code owner December 2, 2025 16:55
Copy link
Member

@billyvg billyvg left a comment

Choose a reason for hiding this comment

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

why is this change needed? couldn't you pass query to the component?

@jerryzhou196
Copy link
Contributor Author

The old ReplaySessionColumn component is hard coded to always sends the user to the replay index page.

Refactored ReplayPreviewPlayer and ReplayTable components to replace 'linkQuery' with 'to' for constructing navigation paths. This change improves code consistency and aligns with recent updates in replay-related components.
@jerryzhou196 jerryzhou196 requested a review from a team December 2, 2025 17:41
@jerryzhou196 jerryzhou196 merged commit e651df5 into master Dec 2, 2025
48 checks passed
@jerryzhou196 jerryzhou196 deleted the new-replay-ui-4-support-generic-link-passing branch December 2, 2025 18:37
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 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.

3 participants