Merge branch 'Remove-destroy-function-logic-from-list-model' into 'master'

Remove destroy function logic from list model

See merge request gitlab-org/gitlab!32237

app/assets/javascripts/boards/models/list.js

@@ -83,13 +83,7 @@ class List {
   }
 
   destroy() {
-    const index = boardsStore.state.lists.indexOf(this);
-    boardsStore.state.lists.splice(index, 1);
-    boardsStore.updateNewListDropdown(this.id);
-
-    boardsStore.destroyList(this.id).catch(() => {
-      // TODO: handle request error
-    });
+    boardsStore.destroy(this);
   }
 
   update() {

app/assets/javascripts/boards/stores/boards_store.js

@@ -547,6 +547,15 @@ const boardsStore = {
   destroyList(id) {
     return axios.delete(`${this.state.endpoints.listsEndpoint}/${id}`);
   },
+  destroy(list) {
+    const index = this.state.lists.indexOf(list);
+    this.state.lists.splice(index, 1);
+    this.updateNewListDropdown(list.id);
+
+    this.destroyList(list.id).catch(() => {
+      // TODO: handle request error
+    });
+  },
 
   saveList(list) {
     const entity = list.label || list.assignee || list.milestone;

changelogs/unreleased/Remove-destroy-function-logic-from-list-model.yml

+---
+title: Remove destroy function logic from list model
+merge_request: 32237
+author: nuwe1
+type: other