[Feature flag] Cleanup manage_compliance_frameworks_modals_refactor
requested to merge 389735-feature-flag-cleanup-manage_compliance_frameworks_modals_refactor into master
What does this MR do and why?
Clean up feature flag manage_compliance_frameworks_modals_refactor
and simplify FF related code.
Remove old controller+views for the create/edit forms + related JS.
Screenshots or screen recordings
n/a
How to set up and validate locally
Feature.enable(:manage_compliance_frameworks_modals_refactor)
Feature.disable(:manage_compliance_frameworks_modals_refactor)
- Visit a top level group, with owner access
- Click Settings -> General
- Scroll down and expand Compliance Frameworks
- Add / Edit flows now use modal instead of full page redirects, regardless of whether the feature flag is enabled or not.
and
- Visit a top level group, with owner access
- Click Security -> Compliance report -> Frameworks
- Add / Edit flows from within the report now use modal instead of full page redirects, regardless of whether the feature flag is enabled or not.
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 #389735 (closed)
Edited by Elwyn Benson