Skip to content
GitLab
Next
    • GitLab: the DevOps platform
    • Explore GitLab
    • Install GitLab
    • How GitLab compares
    • Get started
    • GitLab docs
    • GitLab Learn
  • Pricing
  • Talk to an expert
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    Projects Groups Topics Snippets
  • Register
  • Sign in
  • GitLab GitLab
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Issues 56.6k
    • Issues 56.6k
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 1.8k
    • Merge requests 1.8k
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
    • Test cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Terraform modules
    • Model experiments
  • Monitor
    • Monitor
    • Tracing
    • Incidents
  • Analytics
    • Analytics
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GitLab.orgGitLab.org
  • GitLabGitLab
  • Issues
  • #343872
Closed
Open
Issue created Oct 26, 2021 by Max Woolf@mwoolf2️⃣Maintainer

Remove group_merge_request_approval_settings feature flag from codebase

Problem to solve

Feature flags become unnecessary after we change them to default to on. In #285410 (closed) (%14.5) we changed the group_merge_request_approval_settings feature flag to default to on.

Prposal

In 14.6 (or later) we should cleanup and remove this flag from the codebase.

Implementation guide

These are most (all?) of the files that need to be changed:

  • ee/app/models/ee/project.rb
  • ee/app/policies/ee/group_policy.rb
  • ee/app/policies/ee/project_policy.rb
  • ee/app/controllers/ee/projects_controller.rb
  • ee/app/helpers/ee/projects_helper.rb
  • ee/app/helpers/groups/merge_request_approval_settings_helper.rb
  • ee/config/feature_flags/development/group_merge_request_approval_settings_feature_flag.yml:name
  • ee/spec/features/projects/audit_events_spec.rb
  • ee/spec/features/groups/group_settings_spec.rb
  • ee/spec/models/project_spec.rb stub_feature_flags(group_merge_request_approval_settings_feature_flag
  • ee/spec/models/approval_state_spec.rb
  • ee/spec/requests/api/merge_request_approval_settings_spec.rb
  • ee/spec/helpers/projects_helper_spec.rb
  • ee/lib/api/merge_request_approval_settings.rb
  • doc/user/group/index.md
  • doc/user/project/merge_requests/approvals/settings.md

This list was generated using:

  • grep -R group_merge_request_approval_settings app
  • grep -R group_merge_request_approval_settings doc
  • grep -R group_merge_request_approval_settings config
  • grep -R group_merge_request_approval_settings ee
  • grep -R group_merge_request_approval_settings spec
Edited Feb 16, 2022 by Dan Jensen
Assignee
Assign to
Time tracking