Commit b8d94ca5 authored by Marvin Karegyeya's avatar Marvin Karegyeya Committed by Kushal Pandya

Remove moveMultipleIssues logic from list

parent 7d53ceb1
...@@ -119,16 +119,12 @@ class List { ...@@ -119,16 +119,12 @@ class List {
} }
moveMultipleIssues({ issues, oldIndicies, newIndex, moveBeforeId, moveAfterId }) { moveMultipleIssues({ issues, oldIndicies, newIndex, moveBeforeId, moveAfterId }) {
oldIndicies.reverse().forEach(index => {
this.issues.splice(index, 1);
});
this.issues.splice(newIndex, 0, ...issues);
boardsStore boardsStore
.moveMultipleIssues({ .moveListMultipleIssues({
ids: issues.map(issue => issue.id), list: this,
fromListId: null, issues,
toListId: null, oldIndicies,
newIndex,
moveBeforeId, moveBeforeId,
moveAfterId, moveAfterId,
}) })
......
...@@ -675,6 +675,21 @@ const boardsStore = { ...@@ -675,6 +675,21 @@ const boardsStore = {
}); });
}, },
moveListMultipleIssues({ list, issues, oldIndicies, newIndex, moveBeforeId, moveAfterId }) {
oldIndicies.reverse().forEach(index => {
list.issues.splice(index, 1);
});
list.issues.splice(newIndex, 0, ...issues);
return this.moveMultipleIssues({
ids: issues.map(issue => issue.id),
fromListId: null,
toListId: null,
moveBeforeId,
moveAfterId,
});
},
newIssue(id, issue) { newIssue(id, issue) {
return axios.post(this.generateIssuesPath(id), { return axios.post(this.generateIssuesPath(id), {
issue, issue,
......
---
title: Remove moveMultipleIssues logic from issue model
merge_request: 32243
author: nuwe1
type: other
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