[web] Migrate crypto store to shared worker
ClosedPublic
Actions

Authored by michal on Mar 19 2024, 5:56 AM.

Details

Reviewers

kamil
tomek

Commits

rCOMMbe6c43405452: [web] Migrate crypto store to shared worker

Summary

ENG-6655 : Migrate cryptoStore from redux

Previous diffs modified the code so that they use crypto data in shared worker instead of the data in redux. Now we can migrate the crypto accounts from redux to the shared worker.

Depends on D11348

Test Plan

Run migration, make sure that the the data that was in redux matches the data in shared worker. Flow.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

michal created this revision.Mar 19 2024, 5:56 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMar 19 2024, 5:56 AM

Harbormaster completed remote builds in B27580: Diff 38169.Mar 19 2024, 6:29 AM

michal requested review of this revision.Mar 19 2024, 6:29 AM

michal added inline comments.Mar 19 2024, 7:19 AM

web/redux/crypto-store-reducer.js
16–19 ↗	(On Diff #38169)	In these cases the user should get logged out and the database cleanup will be triggered which will clear the crypto store (both the persisted data in db and the in-memory data).

michal added a child revision: D11352: [web] Deprecate non-worker opaque wasm.Mar 19 2024, 7:34 AM

kamil accepted this revision.Mar 19 2024, 8:55 AM

kamil added inline comments.

web/redux/persist.js
265 ↗	(On Diff #38169)	this is because of the fact that at this time `cryptoStore` is not in `AppState` anymore, is that correct?
web/types/worker-types.js
121 ↗	(On Diff #38169)	Would be nice if could add some comment explaining this