Skip to content
GitLab Next
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • GitLab GitLab
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 43,876
    • Issues 43,876
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 1,388
    • Merge requests 1,388
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GitLab.org
  • GitLabGitLab
  • Issues
  • #93
Closed
Open
Created Nov 27, 2015 by Patricio Cano@patricioContributor

Add webhooks for when a merge request is approved

  • Zendesk ticket: https://gitlab.zendesk.com/agent/tickets/12815

Requested feature

Right now we have webhooks for almost all events surrounding the merge request process, but we don;t have anything setup for when a merge request has been approved, thus this feature request.

Concrete questions / Next steps

I think there are two ways in which we could implement this:

  • Fire a webhook every time someone approves
  • Fire a webhook when the MR has reached the minimum number of approvers

I'm leaning for the second alternative, as it would create less noise and, in my opinion, it focuses on the actual event that is relevant for the status of the merge request.

@JobV what do you think of this?

/cc @vsizov

Assignee
Assign to
Time tracking