Skip to content

Change combined banner over limit check

Vijay Hawoldar requested to merge vij-reached-limit-tweak into master

What does this MR do and why?

In https://gitlab.com/gitlab-org/gitlab/-/issues/433034#note_1695175279 we identified that we should be displaying the combined pre-enforcement banner to namespaces that are >= the user limit, rather than those that are > the user limit.

This MR changes the logic to apply the banner to namespaces that have reached the limit rather than those that are over the limit.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

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

Edited by Vijay Hawoldar

Merge request reports

Loading