Skip to content
Snippets Groups Projects

(Part 9) Multiple approval rules - docs

Merged Paul Slaughter requested to merge 1979-fe-part9-docs into 1979-fe-multiple-approval-rules

What does this MR do?

This MR updates the merge_request_approval docs for the new "Multiple approval rules" feature. Please note that it points to a feature branch (not master) and that the changes described in the docs are behind a feature flag (disabled for now).

Related issues

Part of #1979 (closed)

Author's checklist

Review checklist

  • Your team's review (required)
  • PM's review (recommended, but not a blocker)
  • Technical writer's review (required)
  • Merge the EE-MR first, CE-MR afterwards

/label ~Documentation

Edited by Marcia Ramos

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos added 1 deleted label

    added 1 deleted label

  • Awesome, thanks @pslaughter! I've left a few comments, please assign it back to me when done. :wink:

  • Also, could you please update the MR description with the "Documentation" template?

  • Paul Slaughter changed the description

    changed the description

  • Paul Slaughter marked the checklist item Apply the correct labels and milestone as completed

    marked the checklist item Apply the correct labels and milestone as completed

  • Paul Slaughter added 1 commit

    added 1 commit

    • 2c7bb6d5 - Update docs for multiple approval rules

    Compare with previous version

  • Author Maintainer

    Thanks for the quick response @marcia! I replied to your comments and changed the MR description. Back to you! :soccer:

  • assigned to @marcia

  • Marcia Ramos resolved all discussions

    resolved all discussions

  • Marcia Ramos marked the checklist item Correctly apply the product badges and tiers as completed

    marked the checklist item Correctly apply the product badges and tiers as completed

  • Marcia Ramos marked the checklist item Technical writer's review (required) as completed

    marked the checklist item Technical writer's review (required) as completed

  • Thanks @pslaughter! We're good to go. Why have you set the target to a feature branch instead of master? Should I merge it? :thinking:

  • K, checked on Slack. Merging! :rocket:

  • Marcia Ramos approved this merge request

    approved this merge request

  • merged

  • Marcia Ramos mentioned in commit 3a16953c

    mentioned in commit 3a16953c

  • Automatically picked into https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/9534, will merge into 11-8-stable-ee ready for 11.8.0-rc4-ee.

  • Marcia Ramos mentioned in commit cd4284f0

    mentioned in commit cd4284f0

  • GitLab Release Tools Bot removed 1 deleted label

    removed 1 deleted label

  • mentioned in merge request !9534 (merged)

  • Bob Van Landuyt mentioned in merge request !9455 (merged)

    mentioned in merge request !9455 (merged)

  • Mark Chao mentioned in merge request !9618 (merged)

    mentioned in merge request !9618 (merged)

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading