Skip to content

Fixes LimitAlertComponent access check

Vijay Hawoldar requested to merge vij-fix-limit-alert-access-check into master

What does this MR do and why?

Fixes LimitAlertComponent access check

Ensures that the component and those that inherit from it only render for users who are members of the group/project

Refs #431297 (closed)

How to set up and validate locally

  1. Create a public project in a public group
  2. Add a 1+ MiB file to the project repository
  3. Set the repository size limit to 1MiB (Admin -> Settings -> General)
  4. As the user who created the project, view the group and project - you should see a repository limit banner
  5. Login as another user who has not been added to the group/project and view the project - you should see the limit banner
  6. Checkout this branch - you should no longer see the limit banner as the non-member user

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Vijay Hawoldar

Merge request reports