Skip to content

Commit

Permalink
_StoreMixin is removing handles held by a Trackable collection that i…
Browse files Browse the repository at this point in the history
…t should not. When those handles are removed, other grids using the same collection stop getting update notifications.

Fixes #1397
  • Loading branch information
edhager committed Jan 23, 2018
1 parent 7f676bb commit 34e85b8
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions _StoreMixin.js
Original file line number Diff line number Diff line change
Expand Up @@ -178,10 +178,6 @@ define([

options = options || {};

if (this._renderedCollection.tracking) {
this._renderedCollection.tracking.remove();
}

// Remove observer and existing rows so any sub-row observers will be cleaned up
if (this._observerHandle) {
this._observerHandle.remove();
Expand Down

0 comments on commit 34e85b8

Please sign in to comment.