Remove license check approval frontend
What does this MR do and why?
Describe in detail what your merge request does and why.
Remove license check approval frontend
- remove license check feature
- remove docs
- backend being done in a separate MR (https://gitlab.com/gitlab-org/gitlab/-/compare/master...397067-prepare-removal-of-license-check-feature?from_project_id=278964&straight=false)
- License Compliance Policies being removed in Remove license compliance policies tab (!116814 - merged)
- Fix coverage-check popover
Changelog: removed
EE: true
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Page | Before | After |
---|---|---|
/-/settings/merge_requests Merge Request Approvals |
||
/-/merge_requests/:id License-Check Approval |
||
Coverage Check popover |
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
- Navigate to http://gdk.test:3443/mixed-vulns/security-reports-all/-/settings/merge_requests => Merge Request Approvals
- Verify
License check
isn't an option by default - Create an MR and navigate to it
- Verify the MR approval widget does not have
License Check
as an option
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #397067 (closed)
Related to #403207 (closed)
Edited by Alexander Turinske