Skip to content

Conversation

@jvigliotta
Copy link
Collaborator

Closes #201

Updated the historical session indicator to show which sessions are currently being used to filter historical requests.
For 1 session:

  • it's shown in the indicator text
  • it's shown on hover

For more than one session:

  • it's shown on hover

Updated the notification that shows when you select a historical session to also show which sessions are currently being used to filter.

@jvigliotta jvigliotta requested a review from davetsay February 11, 2025 00:37
@jvigliotta jvigliotta requested a review from davetsay February 24, 2025 21:47
Copy link
Collaborator

@davetsay davetsay left a comment

Choose a reason for hiding this comment

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

Nice. Super tiny nit. i think you did away with "1 and 2" and its just "1, 2", but you added the "1 and 2" in the jsdoc.

@sonarqubecloud
Copy link

@davetsay davetsay merged commit eb701bc into main Feb 24, 2025
2 checks passed
@davetsay davetsay deleted the omm-201 branch February 24, 2025 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clearly display the session(s) selected

3 participants