Skip to content

Revert "Merge branch '10395-COAR-phase-2' into 'master'"

Kerri Miller requested to merge revert-5b0099a7 into master

What does this MR do?

https://gitlab.com/gitlab-org/release/tasks/issues/972

!16187 (merged) was is the data migration MR for moving "code owner approval required" off of Project records onto the project's ProtectedRecords. I split it off from the FE work we were doing for the UI changes associated with this project in order to simplify the review process. That MR (!15862 (merged)) has slipped due to last minute FE issues, and as I’m confirming this data migration MR (!16187 (merged)) on staging, I’m realizing that migrating the data certainly works fine, but if we’re not also launching the UI changes to move “code owner approval required” selection to the branches, new projects that are created in the next month would need a second round of migration. Since the data migration isn't critical, I'd like to revert !16187 (merged) and instead merge it back into !15862 (merged).

(In retrospect, we should've either kept it all in one MR, OR been working on a feature branch so our dependancies matched correctly.)

Conformity

Merge request reports