-
Notifications
You must be signed in to change notification settings - Fork 521
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
Monitor tab - Fix monitor tab crashing #946
Conversation
Signed-off-by: nofar9792 <nofar.cohen@logz.io>
Codecov Report
@@ Coverage Diff @@
## main #946 +/- ##
==========================================
- Coverage 95.49% 95.39% -0.10%
==========================================
Files 240 240
Lines 7521 7521
Branches 1888 1832 -56
==========================================
- Hits 7182 7175 -7
- Misses 333 340 +7
Partials 6 6
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be good if someone with react experience can look at this as well.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lookin good
Signed-off-by: nofar9792 <nofar.cohen@logz.io> Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com>
* Added new SPM tracing events tracking Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * Added tests for new SPM events Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * reworked tracking categories & actions names Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * fixed license block dates; moved debounce from tracking module to component Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * Fix monitor tab crashing (#946) Signed-off-by: nofar9792 <nofar.cohen@logz.io> Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * Add repository to package.json (#949) Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * Preparing release v1.23.0 (#947) * Preparing release v1.23.0 Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * Bump github/codeql-action from 1 to 2 (#938) Bumps [github/codeql-action](https://github.com/github/codeql-action) from 1 to 2. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@v1...v2) --- updated-dependencies: - dependency-name: github/codeql-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * Clarify development setup For #952 Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * Use red on scatterplot for traces if any spans have an error=true tag (#951) * Use red on scatterplot for traces if any spans have an error=true tag Signed-off-by: Ed Snible <snible@us.ibm.com> * Used 'yarn prettier' to appease 'yarn lint' Signed-off-by: Ed Snible <snible@us.ibm.com> Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> * Updated sorting events tracking Signed-off-by: VladislavBryukhanov <gfedcba625@gmail.com> Co-authored-by: nofar9792 <nofar.cohen@logz.io> Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com> Co-authored-by: Pavol Loffay <p.loffay@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ed Snible <esnible@acm.org>
Which problem is this PR solving?
Resolves #945
Short description of the changes
if the xDomain is empty we get an error from react-vis lib