Skip to content

Adds DB column for merge request assignee state

Phil Hughes requested to merge ph/343325/requiresAttentionAPI into master

What does this MR do and why?

Adds a new column to merge request assignees that will allow us to set their review state, which for a new feature can be attention_required.

Up migration:

== 20211019153615 AddStateToMergeRequestAssignees: migrating ==================
-- add_column(:merge_request_assignees, :state, :smallint, {:default=>0, :null=>false})
   -> 0.0062s
== 20211019153615 AddStateToMergeRequestAssignees: migrated (0.0062s) =========

Down migration:

== 20211019153615 AddStateToMergeRequestAssignees: reverting ==================
-- remove_column(:merge_request_assignees, :state, :smallint, {:default=>0, :null=>false})
   -> 0.0132s
== 20211019153615 AddStateToMergeRequestAssignees: reverted (0.0157s) =========

#343325 (closed)

Screenshots or screen recordings

How to set up and validate locally

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Phil Hughes

Merge request reports