diff options
author | Douwe Maan <douwe@gitlab.com> | 2015-11-09 11:12:42 +0000 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2015-11-09 11:12:42 +0000 |
commit | 03b12ee5502e56b3cc9b14dfa92722f52ce41bce (patch) | |
tree | cc43d516cf4999d63011c5a4be501cb2e82cbf51 /doc/development/shell_commands.md | |
parent | 9f055126479000764b488cbb8e91733daaca1c36 (diff) | |
parent | 8f2561b193ad39f116655af0789798b45ad906c8 (diff) | |
download | gitlab-ce-03b12ee5502e56b3cc9b14dfa92722f52ce41bce.tar.gz |
Merge branch 'fix-commits-manual-merge' into 'master'
Fix bug where manually merged branches in a MR would end up with an empty diff
This bug manifested in 8.1 with the refactoring of `RefreshService`. Here's what happens:
1. User create a new branch `foo`.
2. User creates a merge request for `foo`.
3. User merges `foo` into `master` by hand.
4. `RefreshService` reloads the merge request. Since `master` is equivalent to `foo`, this results in an empty diff.
5. `RefreshService` then closes the MR.
This wasn't an issue when you use the normal "Accept Merge Request" flow because the act of clicking the button closes the merge request, so step 4 never happens.
Closes #3314
See merge request !1758
Diffstat (limited to 'doc/development/shell_commands.md')
0 files changed, 0 insertions, 0 deletions