Skip to content

Refactor the delete branch and delete protected branches from HAML to vuejs

What does this MR do?

Are there points in the code the reviewer needs to double check?

Probably the deletion of the spinach test to delete a branch, explanation on the diff

Why was this MR needed?

To help correct a bug that I encountered on this MR https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16437, once this is merged, the aforementioned MR will be good for review/merge

Screenshots (if relevant)

Deleting a normal branch

Before After
Screen_Shot_2018-02-05_at_12.25.50_PM Screen_Shot_2018-02-05_at_12.24.17_PM

Deleting a protected branch

Before After
Screen_Shot_2018-02-05_at_12.25.59_PM Screen_Shot_2018-02-05_at_12.24.27_PM

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #42603 (moved)

Edited by Jose Ivan Vargas

Merge request reports