Skip to content

Updated the project_sidebar_refactor flag

Denys Mishunov requested to merge dmishunov-scoped-feature-flag into master

What does this MR do?

  1. Renames the flag to sidebar_refactor to reflect the wider scope of the changes covered by the flag (it won't be only about the projects)
  2. Scoped the flag to the users to have granular control over the rollout strategy
  3. Fixes the issue introduced in !58638 (merged) where the flyout menu wasn't shown for the "Requirements" item at all.

Testing

In the Rails console (rails c), enable the sidebar_refactor feature flag for your current user. The Requirements menu in the lefthand navigation should now behave exactly as "merge Requests"

Screenshots (strongly suggested)

Without the flag With the flag enabled for the current user
Expanded menu (:hover state) Screenshot_2021-04-23_at_12.42.58 Screenshot_2021-04-23_at_12.37.52
Collapsed menu (:hover state) Screenshot_2021-04-23_at_12.42.49 Screenshot_2021-04-23_at_12.37.39

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

References #323178 (closed)

Merge request reports