-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Missing Object] Notifications are shown as minimized #6416
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6416 +/- ##
==========================================
- Coverage 54.98% 54.65% -0.33%
==========================================
Files 620 620
Lines 26490 26494 +4
Branches 2392 2392
==========================================
- Hits 14565 14480 -85
- Misses 11281 11367 +86
- Partials 644 647 +3
*This pull request uses carry forward flags. Click here to find out more.
... and 19 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
…nto bugfix/issue-6349
…ssing/type unknown
… using the telemetry API under the hood and add check for historical data
@@ -108,6 +108,24 @@ describe('The Notifiation API', () => { | |||
}); | |||
}); | |||
|
|||
describe('the error method notificiation', () => { |
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.
At some point we should add a test to see that if a new notification comes in while the notifications are being viewed, that they don't end up being minimized.
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.
Looks good to me!
Closes #6349
Describe your changes:
Adds the ability to pass
minimized: true
to error notifications show those notifications as minimized. Also shows missing object error notifications as minimized by default.All Submissions:
Author Checklist
Reviewer Checklist