Add milestones to roadmap

Btw, the title of the MR stills says WIP, you might want to remove that, as it is ready for review now. And probably you should change the milestone to 12.9, since 12.8 finished a couple of days ago.

approved this merge request

assigned to @godfat and unassigned @alexbuijs

approved this merge request

unassigned @godfat

unmarked as a Work In Progress

changed milestone to %12.9

added missed:12.8 label

added workflowin review label and removed workflowready for development label

mentioned in merge request !25774 (merged)

mentioned in merge request !25778 (merged)

added feature flag label

@fguibert As an aside, if you find that bootstrap doesn't have something, you can always add a new utility/common class in app/assets/stylesheets/framework/common.scss or utilities.scss

added 7 commits

4a2be2ac - Show milestones in Roadmap
f0a759aa - Add Milestones to Roadmap
87dff197 - Add milestones to roadmap
82a217a0 - Add milestones to roadmap
9d036d20 - Add Milestones in Roadmap
e451de6e - Add Milestones in Roadmap
00a5b9e8 - Add Milestones in Roadmap

Compare with previous version

added 1 commit

10b93f00 - Add Milestones in Roadmap

Compare with previous version

added 487 commits

10b93f00...8705340a - 478 commits from branch master
16a8c5f3 - Add milestones to roadmap - GraphQL mock milestones - vuex state
ca2da0d2 - Show milestones in Roadmap
6dfb9b6b - Show milestones in Roadmap
0a20549d - Add Milestones to Roadmap
e0b2cf36 - Add milestones to roadmap
5a8e74ac - Add milestones to roadmap
d9aa85c5 - Add Milestones in Roadmap
81d3db31 - Add Milestones in Roadmap
67c6570e - Add Milestones in Roadmap

Compare with previous version

added 170 commits

67c6570e...a01fb250 - 161 commits from branch master
8db374f6 - Add milestones to roadmap - GraphQL mock milestones - vuex state
56561751 - Show milestones in Roadmap
8c635153 - Show milestones in Roadmap
92692be1 - Add Milestones to Roadmap
5c88ac4b - Add milestones to roadmap
09453522 - Add milestones to roadmap
c515a70f - Add Milestones in Roadmap
6e058bac - Add Milestones in Roadmap
f68f0b35 - Add Milestones in Roadmap

Compare with previous version

added 1 commit

11424b35 - Add Milestones in Roadmap

Compare with previous version

added 200 commits

11424b35...abe7a9ba - 191 commits from branch master
f5375e27 - Add milestones to roadmap - GraphQL mock milestones - vuex state
dd683e5e - Show milestones in Roadmap
1cb0d9c8 - Show milestones in Roadmap
dc0778aa - Add Milestones to Roadmap
c495a01c - Add milestones to roadmap
1702a409 - Add milestones to roadmap
69f2b2ff - Add Milestones in Roadmap
cd72d2b1 - Add Milestones in Roadmap
792240ca - Add Milestones in Roadmap

Compare with previous version

added 1 commit

84c5c06c - Add Milestones in Roadmap

Compare with previous version

added 1 commit

154ea8a9 - Add Milestones in Roadmap

Compare with previous version

added 1 commit

81c81ea0 - Add Milestones in Roadmap

Compare with previous version

added database databasereview pending labels

added 191 commits

81c81ea0...dd1aedef - 182 commits from branch master
7b504f7b - Add milestones to roadmap - GraphQL mock milestones - vuex state
c9789691 - Show milestones in Roadmap
5fcfc308 - Show milestones in Roadmap
d9c6e078 - Add Milestones to Roadmap
2f3fb6e6 - Add milestones to roadmap
a7a04ac1 - Add milestones to roadmap
39ed6d6f - Add Milestones in Roadmap
f3150002 - Add Milestones in Roadmap
052b5104 - Add Milestones in Roadmap

Compare with previous version

@kushalpandya Hey Kushal! Could you please give this MR a look? I've already reviewed most of it, and there are some unresolved discussions for you to take a look at. I will finish my review (mostly left surrounding optimizing CSS) as soon as I can on Monday. Thank you!

assigned to @kushalpandya

unassigned @kushalpandya

added 1 commit

cc795fc8 - Add Milestones in Roadmap

Compare with previous version

assigned to @kushalpandya

unassigned @kushalpandya

@uhlexsis There are a few UX questions here about following our system guidelines. Following our initial conversation about this feature, I customised the CSS to squish how we display milestones as we are not initially stacking them. Could you enlighten us on some questions here and give this a UX review? Thanks

assigned to @uhlexsis

added 1 commit

e5a3f4b8 - Add Milestones in Roadmap

Compare with previous version

added 1 commit

485e1205 - Add Milestones in Roadmap

Compare with previous version

added 1 commit

5a381c8b - Add Milestones in Roadmap

Compare with previous version

@fguibert Let me know when this is ready and I'll do the approval :) I will unassign myself for now.

unassigned @lauraMon

assigned to @lauraMon

approved this merge request

unassigned @lauraMon

added 1 commit

b3b0d5e8 - Add Milestones in Roadmap

Compare with previous version

removed database databasereview pending labels

added 1 commit

40943cd5 - Add Milestones in Roadmap

Compare with previous version

added 1 commit

38dc3b0b - Add Milestones in Roadmap

Compare with previous version

	2 Warnings
	This merge request is quite big (more than 1549 lines changed), please consider splitting it into multiple merge requests.
	You have edited karma spec files. Please consider migrating them to jest.

Edited karma files

You have edited the following karma spec files. Please consider migrating them to jest:

ee/spec/javascripts/roadmap/components/epic_item_spec.js
ee/spec/javascripts/roadmap/components/roadmap_app_spec.js
ee/spec/javascripts/roadmap/components/roadmap_shell_spec.js
ee/spec/javascripts/roadmap/mixins/months_preset_mixin_spec.js
ee/spec/javascripts/roadmap/mixins/quarters_preset_mixin_spec.js
ee/spec/javascripts/roadmap/mixins/weeks_preset_mixin_spec.js
ee/spec/javascripts/roadmap/mock_data.js
ee/spec/javascripts/roadmap/store/actions_spec.js
ee/spec/javascripts/roadmap/store/mutations_spec.js
ee/spec/javascripts/roadmap/utils/roadmap_item_utils_spec.js

In order to align with our Iteration value, migration can also be done as a follow-up.

For more information: Jestodus epic

Reviewer roulette

Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category (e.g. frontend or backend), and by a maintainer in all other categories.

To spread load more evenly across eligible reviewers, Danger has randomly picked a candidate for each review slot. Feel free to override this selection if you think someone else would be better-suited, or the chosen person is unavailable.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines.

Once you've decided who will review this merge request, mention them as you normally would! Danger does not (yet?) automatically notify them for you.