Use provide/inject for groupFullPath instead of props
What does this MR do?
For the group-level vulnerability report and security dashboard:
A groupFullPath
property is passed in by the backend, then used throughout the frontend for various queries. Currently, this property is passed down multiple levels in Vue through props. This MR changes it to use provide/inject instead, which is slightly cleaner than using props.
The component tree looks like this:
The *-init.js
files gets groupFullPath
from el.dataset
, then passes it to dashboard.vue
/security_charts.vue
, which is then passed down again to their sub-components.
Does this MR meet the acceptance criteria?
Conformity
-
📋 Does this MR need a changelog?-
I have included a changelog entry. -
I have not included a changelog entry because _____.
-
- [-] Documentation (if required)
-
Code review guidelines -
Merge request performance guidelines - [-] Style guides
- [-] Database guides
-
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. -
Tested in all supported browsers - [-] Informed Infrastructure department of a default or new setting change, if applicable per definition of done
Edited by Daniel Tian