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
fix: change Alert Permissions #16118
Conversation
5b8c2fc
to
f4c5326
Compare
Codecov Report
@@ Coverage Diff @@
## master #16118 +/- ##
==========================================
- Coverage 76.84% 76.84% -0.01%
==========================================
Files 995 995
Lines 52886 52885 -1
Branches 6721 6721
==========================================
- Hits 40640 40638 -2
- Misses 12021 12022 +1
Partials 225 225
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@eschutho For public users (which is the starter tier) are we turning off their Alerts_Reports feature flag or is that permissions based? |
🏷 2021.31 |
* added google alert * reworked permissions (cherry picked from commit 606a7bf)
* added google alert * reworked permissions (cherry picked from commit 606a7bf)
* added google alert * reworked permissions
* added google alert * reworked permissions (cherry picked from commit 606a7bf)
* added google alert * reworked permissions
* added google alert * reworked permissions (cherry picked from commit 606a7bf)
* added google alert * reworked permissions (cherry picked from commit 35e7a32)
SUMMARY
This PR adjusts who can see the new dashboard and chart subscribe Icon. Previously it was based on who can add a report, but now it is based on if you have access to alerts and reports in the settings menu.
While looking through this, I noticed that a limited/gamma user does have permissions to add, view, and has menu access to Alerts and Reports. Maybe this is an oversight and we need to adjust the permissions for Gamma users?
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Admin User:
Gamma:
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION