This differential was created in response to the issue pointed by Ashoat: https://phab.comm.dev/D9532#inline-60290. Cryptographically, it doesn't make
sense in our project to have primaryAccount set but nulled notificationAccount. Additionally it doesn't make sense if there is primaryAccount but there are
no primaryIdentityKeys. That said we need one redux action to set entire CryptoStore at once and this part of redux should either have all fields set all be null.
Details
Details
Log-in and Log-out while observing Redux extension of Developer Tools. Ensure that LOG_OUT_SUCCESS action succesfully changes CryptoStore to null and
shortly SET_CRYPTO_STORE action is dispatched that correctly sets entire CryptoStore.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
web/selectors/tunnelbroker-selectors.js | ||
---|---|---|
11 ↗ | (On Diff #33556) | My mistake indeed. This issue didn't occur during my testing. |