Notify subscribers on the next tick

[alex-mechler]

cc @kidroca @marcaaron

Details

This updates subscribers of data changes on the next tick, rather than blocking on notifying subscribers.

Related Issues

Expensify/App#3485 (comment)
Expensify/App#3488 (comment)

Automated Tests

Existing automated tests cover these changes.

Linked PRs

Expensify/App#3535

[Julesssss]

Resolves the endless spinner issue on login. Even with the redirect to /r, this seems like a good improvement to me.

[marcaaron]

Looks good, but curious if we want to also make a similar change for merge(), multiSet() and mergeCollection() ? There are a few other methods that set data and also were calling keyChanged() or keysChanged() async before kid roca's changes.

[alex-mechler]

Ah good call Marc, I can test adding those. That might solve some of the other issues I am seeing.

[alex-mechler]

Updated to notify the subscribers on the next tick for remove and multiset to match behavior from prior to the cache.

I did not notice any difference, however this better matches the prior behavior, and passes all tests.