Roadmaps refactoring: move non-reactive properties from Vuex to inject
What does this MR do and why?
This MR is a part of refactoring roadmaps to use VueApollo effort. Here, before introducing VueApollo, we reduce the number of Vuex properties by moving a few of them to provide/inject
.
Two of these properties, epicIid
and fullPath
are not removed from Vuex yet (this will be done in subsequent MRs) but added to provided properties because we will need them in the next MR as variables for GraphQL calls from the component.
MRs for Roadmaps refactoring
Title | Link |
---|---|
|
!148520 (merged) |
Move fetching child epics logic to epic_item.vue
|
TBD |
Move fetching milestones logic | TBD |
Move fetching group epics logic | TBD |
Move filter params to root component property | TBD |
Move local settings to Apollo cache | TBD |
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.