Skip to content

Commit 6dba2c8

Browse files
benvinegarmattrobenolt
authored andcommitted
Merge pull request #2608 from getsentry/fix-merge-clear
Fix merge success not clearing rows from stream UI
1 parent 00e09b4 commit 6dba2c8

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/sentry/static/sentry/app/stores/groupStore.jsx

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -258,13 +258,17 @@ const GroupStore = Reflux.createStore({
258258
},
259259

260260
onMerge(changeId, itemIds) {
261+
itemIds = this._itemIdsOrAll(itemIds);
262+
261263
itemIds.forEach(itemId => {
262264
this.addStatus(itemId, 'merge');
263265
});
264266
this.trigger(new Set(itemIds));
265267
},
266268

267269
onMergeError(changeId, itemIds, response) {
270+
itemIds = this._itemIdsOrAll(itemIds);
271+
268272
itemIds.forEach(itemId => {
269273
this.clearStatus(itemId, 'merge');
270274
});
@@ -273,6 +277,8 @@ const GroupStore = Reflux.createStore({
273277
},
274278

275279
onMergeSuccess(changeId, mergedIds, response) {
280+
mergedIds = this._itemIdsOrAll(mergedIds); // everything on page
281+
276282
mergedIds.forEach(itemId => {
277283
this.clearStatus(itemId, 'merge');
278284
});

0 commit comments

Comments
 (0)