FE: Move the threat monitoring policies page to Policies sidebar
Why are we doing this work
- Threat Monitoring is being updated to be more specific to the monitoring aspects (i.e. statistics, alerts, etcetera) * Additionally, the
Policiestab is becoming much larger and complicated as it will be handling many types of policies - Take over the current
Security & Compliance => Scan Policiessidebar route and move thePoliciestab into it and rename itSecurity & Compliance => Policies - match all existing functionality of the current
Policiestab
Relevant links
Design:
| Link | Page | Page w/ modal |
|---|---|---|
| #322473 (closed) | ![]() |
![]() |
Non-functional requirements
-
Documentation -
Testing
Implementation plan
-
frontend Show the Policy List component, on the Security & Compliance => Scan Policiessidebar route by updating the security_policies_list.js (if this is a lot of work, one could create a duplicate security_policies_list.js file, make the changes there, and then conditionally render this file in the policies/show/index file based on the feature flag). Do NOT remove the Policies tab fromSecurity & Compliance => Threat Monitoringbecauue otherwise we will be removing the policy tab without giving all users access to the Scan Policies sidebar, which is hidden behind the:security_orchestration_policies_configurationfeature flag; we must wait until this feature flag is enabled by default and removed before removing the policy tab (MR: !67419 (merged)) -
frontend move the Security Policy Project Selector into a modal that is openable on the Security & Compliance => Scan Policiespage (MR: !67035 (merged)) -
backend update the more informationlink to https://docs.gitlab.com/ee/user/application_security/policies/ (MR: !67496 (merged)) -
frontend/backend create a newfolder/index in the policies route folder load the policy editor component when the user clicks theNew Policybutton and move new route over topolicies_controller. update the breadcrumb fornewpolicy page -
backend Rename the Security & Compliance => Scan PoliciestoSecurity & Compliance => Policies(MR: !67421 (merged))
Edited by Alexander Turinske

