Commit a6eaa51c authored by Clement Ho's avatar Clement Ho

Merge branch '32030-transient-failure-in-spec-features-boards-boards_spec-rb' into 'master'

Fix transient job failures relating to "unhandled Promise rejection" in issue boards

Closes #32030

See merge request !11222
parents b2fc6326 fb7c9401
...@@ -35,7 +35,10 @@ gl.issueBoards.Board = Vue.extend({ ...@@ -35,7 +35,10 @@ gl.issueBoards.Board = Vue.extend({
filter: { filter: {
handler() { handler() {
this.list.page = 1; this.list.page = 1;
this.list.getIssues(true); this.list.getIssues(true)
.catch(() => {
// TODO: handle request error
});
}, },
deep: true, deep: true,
}, },
......
...@@ -70,7 +70,10 @@ export default { ...@@ -70,7 +70,10 @@ export default {
list.id = listObj.id; list.id = listObj.id;
list.label.id = listObj.label.id; list.label.id = listObj.label.id;
list.getIssues(); list.getIssues()
.catch(() => {
// TODO: handle request error
});
}); });
}) })
.catch(() => { .catch(() => {
......
...@@ -90,7 +90,10 @@ export default { ...@@ -90,7 +90,10 @@ export default {
if (this.scrollHeight() <= this.listHeight() && if (this.scrollHeight() <= this.listHeight() &&
this.list.issuesSize > this.list.issues.length) { this.list.issuesSize > this.list.issues.length) {
this.list.page += 1; this.list.page += 1;
this.list.getIssues(false); this.list.getIssues(false)
.catch(() => {
// TODO: handle request error
});
} }
if (this.scrollHeight() > Math.ceil(this.listHeight())) { if (this.scrollHeight() > Math.ceil(this.listHeight())) {
......
...@@ -108,6 +108,8 @@ gl.issueBoards.IssuesModal = Vue.extend({ ...@@ -108,6 +108,8 @@ gl.issueBoards.IssuesModal = Vue.extend({
if (!this.issuesCount) { if (!this.issuesCount) {
this.issuesCount = data.size; this.issuesCount = data.size;
} }
}).catch(() => {
// TODO: handle request error
}); });
}, },
}, },
......
...@@ -25,7 +25,9 @@ class List { ...@@ -25,7 +25,9 @@ class List {
} }
if (this.type !== 'blank' && this.id) { if (this.type !== 'blank' && this.id) {
this.getIssues(); this.getIssues().catch(() => {
// TODO: handle request error
});
} }
} }
...@@ -52,11 +54,17 @@ class List { ...@@ -52,11 +54,17 @@ class List {
gl.issueBoards.BoardsStore.state.lists.splice(index, 1); gl.issueBoards.BoardsStore.state.lists.splice(index, 1);
gl.issueBoards.BoardsStore.updateNewListDropdown(this.id); gl.issueBoards.BoardsStore.updateNewListDropdown(this.id);
gl.boardService.destroyList(this.id); gl.boardService.destroyList(this.id)
.catch(() => {
// TODO: handle request error
});
} }
update () { update () {
gl.boardService.updateList(this.id, this.position); gl.boardService.updateList(this.id, this.position)
.catch(() => {
// TODO: handle request error
});
} }
nextPage () { nextPage () {
...@@ -146,11 +154,17 @@ class List { ...@@ -146,11 +154,17 @@ class List {
this.issues.splice(oldIndex, 1); this.issues.splice(oldIndex, 1);
this.issues.splice(newIndex, 0, issue); this.issues.splice(newIndex, 0, issue);
gl.boardService.moveIssue(issue.id, null, null, moveBeforeIid, moveAfterIid); gl.boardService.moveIssue(issue.id, null, null, moveBeforeIid, moveAfterIid)
.catch(() => {
// TODO: handle request error
});
} }
updateIssueLabel(issue, listFrom, moveBeforeIid, moveAfterIid) { updateIssueLabel(issue, listFrom, moveBeforeIid, moveAfterIid) {
gl.boardService.moveIssue(issue.id, listFrom.id, this.id, moveBeforeIid, moveAfterIid); gl.boardService.moveIssue(issue.id, listFrom.id, this.id, moveBeforeIid, moveAfterIid)
.catch(() => {
// TODO: handle request error
});
} }
findIssue (id) { findIssue (id) {
......
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