Skip to content

Group by cadence in iteration dropdown widgets [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Coung Ngo requested to merge 293925-iteration-dropdown-group-by-cadence into master

What does this MR do?

Iteration cadences &5077 are being introduced behind the feature flag :iteration_cadence which is off by default.

This MR updates the iteration dropdown widgets to group by cadence in the:

  • Issue sidebar
  • Group boards drawer
  • Project boards drawer
  • Group bulk edit sidebar
  • Project bulk edit sidebar

Implementation notes:

  • A new separate iteration dropdown component IterationSidebarDropdownWidget was created which reuses SidebarDropdownWidget. This is so we don't have iteration-only logic in the generic SidebarDropdownWidget component.
  • The :iteration_cadences feature flag will be removed after the release, so the conditionals were added in a way that's easy to delete.
  • The feature flag yml file iteration_cadences.yml was moved from EE to CE because we're using the feature flag in CE controllers

Issue: #293925 (closed)

Screenshots (strongly suggested)

:iteration_cadences FF off :iteration_cadences FF on
Issue sidebar h5bp/html5-boilerplate/-/issues/1 off-issue on-issue
Group board groups/h5bp/-/boards off-group-board on-group-board
Project board h5bp/html5-boilerplate/-/boards/3 off-project-board on-project-board
Group bulk edit groups/h5bp/-/issues off-group-bulk on-group-bulk
Project bulk edit h5bp/html5-boilerplate/-/issues off-project-bulk on-project-bulk

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #293925 (closed)

Edited by Coung Ngo

Merge request reports