-
Notifications
You must be signed in to change notification settings - Fork 498
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
Ignore the voice broadcast chunks at the notifications level #7230
Conversation
Codecov ReportBase: 11.83% // Head: 11.83% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #7230 +/- ##
===========================================
- Coverage 11.83% 11.83% -0.01%
===========================================
Files 1629 1630 +1
Lines 160157 160252 +95
Branches 65390 65439 +49
===========================================
+ Hits 18962 18973 +11
- Misses 140558 140641 +83
- Partials 637 638 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
4089b62
to
d620705
Compare
d8697a8
to
3b6639b
Compare
3b6639b
to
16c03c9
Compare
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.
LGTM, but I cannot test it.
@phlniji you could test on a physical device with element target. |
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.
LGTM, I don't have any VB notifications anymore.
44d2b4b
to
0ff9c94
Compare
Kudos, SonarCloud Quality Gate passed! |
No description provided.