Follow-up from "Migrates LabelsSelect to LabelsSelectWidget in TestCaseSidebar and fixes bug"
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
The following discussion from !109527 (merged) should be addressed:
-
@rcrespo3 started a discussion: (+2 comments) @fguibert The proper architecture should actually be as follows.
- Duplicate
issuableLabelsQueriesobject intoee/app/assets/javascripts/sidebar/constants.js - Since it's in the
eefolder, it shouldn't be a problem to importeequeries there. - Update
issuableLabelsQueriesimport inapp/assets/javascripts/sidebar/components/labels/labels_select_widget/labels_select_root.vueto useee_else_ceso the appropriate version gets loaded.
Note that epics query should follow this architecture and the existing issuableLabelsQueries object should only contain queries for issues and merge requests. A possible way to approach this is to follow the easy solution with the existing architecture in this MR and fix the ce/ee separation in a follow up MR.
Edited by 🤖 GitLab Bot 🤖