search

getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
39.22k stars 4.21k forks source link

fix(explore): Force chart type when known #81109

Closed Zylphrex closed 8 hours ago

Zylphrex commented 18 hours ago

In regular mode, the chart type can be inferred via the series name. This doesn't work in top N mode when the series name are the table values. In that case, we know there's only 1 type so force the chart type.

codecov[bot] commented 18 hours ago

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

:white_check_mark: All tests successful. No failed tests found.

Files with missing lines Patch % Lines
static/app/views/explore/charts/index.tsx 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #81109 +/- ## ======================================= Coverage 78.48% 78.48% ======================================= Files 7215 7215 Lines 319812 319813 +1 Branches 44045 44045 ======================================= + Hits 251005 251007 +2 + Misses 62416 62415 -1 Partials 6391 6391 ```