Skip to content
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(sessions): tutorials and getting started guides for sessions in the UI" #5593

Closed
wants to merge 20 commits into from

Conversation

mikeldking
Copy link
Contributor

RogerHYang and others added 20 commits December 2, 2024 11:48
* feat(sesions): add session details page

* remove unused import, add error boundary

* pull user from root span
* feat(sessions): add trace latency p50 to session details

* fix type for latency in ui

* update test, fix where clause

* add is not none check for db filter

* revert latency dataloader changes

* revert dataloader changes for project span and trace latency

* refactor to use its own data loader

* remove extra param from test

* add unit tests

* ruff

* fix import

* fix name filter in other dataloader tests

* update unit test fixture naming, add unit test for project session graphql trace_latency_ms_quantile field

* clean up imports

* pin aiohttp

* fix unit test deps
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Dec 3, 2024
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@mikeldking mikeldking closed this Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants