Commit e173db9c authored by Filipa Lacerda's avatar Filipa Lacerda

Merge branch 'ide-open-empty-merge-request' into 'master'

Allow the Web IDE to open empty merge requests

Closes #48166

See merge request gitlab-org/gitlab-ce!21102
parents a21906a4 b401bfd0
...@@ -117,7 +117,7 @@ router.beforeEach((to, from, next) => { ...@@ -117,7 +117,7 @@ router.beforeEach((to, from, next) => {
mergeRequestId: to.params.mrid, mergeRequestId: to.params.mrid,
}) })
.then(mr => { .then(mr => {
store.dispatch('updateActivityBarView', activityBarViews.review); store.dispatch('setCurrentBranchId', mr.source_branch);
store.dispatch('getBranchData', { store.dispatch('getBranchData', {
projectId: fullProjectId, projectId: fullProjectId,
...@@ -144,6 +144,10 @@ router.beforeEach((to, from, next) => { ...@@ -144,6 +144,10 @@ router.beforeEach((to, from, next) => {
}), }),
) )
.then(mrChanges => { .then(mrChanges => {
if (mrChanges.changes.length) {
store.dispatch('updateActivityBarView', activityBarViews.review);
}
mrChanges.changes.forEach((change, ind) => { mrChanges.changes.forEach((change, ind) => {
const changeTreeEntry = store.state.entries[change.new_path]; const changeTreeEntry = store.state.entries[change.new_path];
......
...@@ -54,9 +54,6 @@ export const setFileActive = ({ commit, state, getters, dispatch }, path) => { ...@@ -54,9 +54,6 @@ export const setFileActive = ({ commit, state, getters, dispatch }, path) => {
commit(types.SET_FILE_ACTIVE, { path, active: true }); commit(types.SET_FILE_ACTIVE, { path, active: true });
dispatch('scrollToTab'); dispatch('scrollToTab');
commit(types.SET_CURRENT_PROJECT, file.projectId);
commit(types.SET_CURRENT_BRANCH, file.branchId);
}; };
export const getFileData = ({ state, commit, dispatch }, { path, makeFileActive = true }) => { export const getFileData = ({ state, commit, dispatch }, { path, makeFileActive = true }) => {
......
---
title: Fix empty merge requests not opening in the Web IDE
merge_request: 21102
author:
type: fixed
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment