Commit f2ba0cde authored by Florie Guibert's avatar Florie Guibert

Roadmap settings - Include subgroup in group milestones

Behind `roadmap_settings` feature flag
parent 5a018bed
...@@ -48,7 +48,7 @@ export default { ...@@ -48,7 +48,7 @@ export default {
milestonesToShow() { milestonesToShow() {
switch (this.milestonesType) { switch (this.milestonesType) {
case MILESTONES_GROUP: case MILESTONES_GROUP:
return this.milestones.filter((m) => m.groupMilestone && !m.subgroupMilestone); return this.milestones.filter((m) => m.groupMilestone);
case MILESTONES_SUBGROUP: case MILESTONES_SUBGROUP:
return this.milestones.filter((m) => m.subgroupMilestone); return this.milestones.filter((m) => m.subgroupMilestone);
case MILESTONES_PROJECT: case MILESTONES_PROJECT:
......
...@@ -273,7 +273,7 @@ RSpec.describe 'group epic roadmap', :js do ...@@ -273,7 +273,7 @@ RSpec.describe 'group epic roadmap', :js do
it 'renders milestones based on filter' do it 'renders milestones based on filter' do
milestones_counts = { milestones_counts = {
'Show all milestones' => 4, 'Show all milestones' => 4,
'Show group milestones' => 1, 'Show group milestones' => 2, # group & subgroup
'Show sub-group milestones' => 1, 'Show sub-group milestones' => 1,
'Show project milestones' => 2 'Show project milestones' => 2
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment