-
Notifications
You must be signed in to change notification settings - Fork 95
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
Threshold warning for content composer #1852
Conversation
Codecov Report
@@ Coverage Diff @@
## gsa-8.0 #1852 +/- ##
===========================================
- Coverage 40.04% 40.03% -0.01%
===========================================
Files 976 977 +1
Lines 22464 22473 +9
Branches 6326 6306 -20
===========================================
+ Hits 8995 8997 +2
- Misses 12184 12190 +6
- Partials 1285 1286 +1
Continue to review full report at Codecov.
|
e0d7010
to
cf5e518
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.
I would move thresholdmessage.js to web/pages/reports too
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! Thanks a lot
Checklist: