Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[dashboard] Use user site permissions for dashboard charts #8132

Merged
merged 2 commits into from
Dec 1, 2022
Merged

[dashboard] Use user site permissions for dashboard charts #8132

merged 2 commits into from
Dec 1, 2022

Conversation

CamilleBeau
Copy link
Contributor

Brief summary of changes

This PR uses the user sites instead of a full site list in the visualizations of statistics displayed on the dashboard.

  • Have you updated related documentation?

Testing instructions (if applicable)

  1. Test the dashboard charts with a user who only has access to some but not all sites
  2. Make sure that the numbers are correct
  3. Repeat steps 1 and 2 but with a user who has all sites
  4. Repeat steps 1 and 2 but with a user who has the "all sites" permission

@CamilleBeau CamilleBeau force-pushed the 2022_07_05_Dashboard_Site_separation branch from 260bd17 to 20b7185 Compare November 30, 2022 17:03
@driusan driusan merged commit 6eb61c5 into aces:main Dec 1, 2022
@ridz1208 ridz1208 added this to the 25.0.0 milestone Mar 6, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@maltheism maltheism maltheism approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
25.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@CamilleBeau @maltheism @driusan @ridz1208