Skip to content

MR Approvals Frontend-Backend Integration

username-removed-408230 requested to merge approval-integration into master

@jneen's Todos:

  • create notification email for unapprove
  • delete deprecated controller actions and associated views

@brycepj Todos:

  • Check/fix lint errors and failing specs
  • Remove exclamation marks for both emails
  • "Merge request !xyz was approved by"
  • Update cross icon and merge request icon
  • Use @approved_by and @unapproved_by

Closes #1262 (closed), #894 (closed), #1287 (closed), and #1288 (closed) (once notification email is done)

Merge request reports