Add standalone finding modal component
What does this MR do and why?
Currently, ee/app/assets/javascripts/vue_shared/security_reports/components/modal.vue
is used to show the info modal for a finding on the pipeline security tab and the MR security widget. This component expects that the parent component uses Vuex, but we're moving away from Vuex. This MR adds a finding_modal.vue
component that wraps modal.vue
and moves the Vuex code into this component, in order to break the Vuex dependency.
Note that for this MR finding_modal.vue
is read only, and the actions are all noop functions that will be added in later MRs.
How to set up and validate locally
- Enable the
standalone_finding_modal
feature flag. - Run a pipeline that has security vulnerabilities, if you haven't already.
- Go to the pipeline and click on the security tab.
- Click on any finding in the list to bring up the modal.
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 #382913 (closed)