Merge remote-tracking branch 'origin/develop' into develop
This commit is contained in:
commit
107f83f84f
1 changed files with 7 additions and 0 deletions
|
@ -48,6 +48,9 @@ class FlairStore extends EventEmitter {
|
|||
// reject: () => {}
|
||||
// }
|
||||
};
|
||||
this._usersInFlight = {
|
||||
// This has the same schema as _usersPending
|
||||
};
|
||||
|
||||
this._debounceTimeoutID = null;
|
||||
}
|
||||
|
@ -125,12 +128,16 @@ class FlairStore extends EventEmitter {
|
|||
} catch (err) {
|
||||
// Propagate the same error to all usersInFlight
|
||||
Object.keys(this._usersInFlight).forEach((userId) => {
|
||||
// The promise should always exist for userId, but do a null-check anyway
|
||||
if (!this._usersInFlight[userId]) return;
|
||||
this._usersInFlight[userId].reject(err);
|
||||
});
|
||||
return;
|
||||
}
|
||||
const updatedUserGroups = resp.users;
|
||||
Object.keys(this._usersInFlight).forEach((userId) => {
|
||||
// The promise should always exist for userId, but do a null-check anyway
|
||||
if (!this._usersInFlight[userId]) return;
|
||||
this._usersInFlight[userId].resolve(updatedUserGroups[userId] || []);
|
||||
});
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue