Page MenuHomePhabricator
Differential D7070

[web] Add `CryptoStore` to `persistConfig.whitelist`
ClosedPublic
Actions

Authored by atul on Tue, Mar 14, 2:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 18, 9:37 AM
Unknown Object (File)
Thu, Mar 16, 7:04 PM
Unknown Object (File)
Thu, Mar 16, 2:27 AM
Unknown Object (File)
Wed, Mar 15, 8:42 PM
Unknown Object (File)
Wed, Mar 15, 8:41 PM
Unknown Object (File)
Wed, Mar 15, 4:06 PM
Subscribers
kamil
tomek

Details

Reviewers
ashoat
marcin
kamil
Commits
rCOMM6bfaf0746b8f: [web] Add `CryptoStore` to `persistConfig.whitelist`
Summary

We were previously excluding CryptoStore from redux-persist as we were working to avoid having to create a ton of migrations.

But now that we have primaryAccount/notificationAccount and primaryIdentityKeys/notificationIdentityKeys I think things are stable in the short term and we can start persisting.

Depends on D7081

Test Plan

Reload page and see that cryptoStore is included in REHYDRATE action:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Tue, Mar 14, 2:28 PM
Herald added a subscriber: tomek. · View Herald TranscriptTue, Mar 14, 2:28 PM
Harbormaster completed remote builds in B17333: Diff 23740.Tue, Mar 14, 2:43 PM
atul requested review of this revision.Tue, Mar 14, 2:43 PM
ashoat accepted this revision.Tue, Mar 14, 6:36 PM
This revision is now accepted and ready to land.Tue, Mar 14, 6:36 PM
ashoat added a reviewer: kamil.Tue, Mar 14, 6:36 PM
ashoat added a subscriber: kamil.

FYI @kamil, another thing we'll need to move to SQLite

atul edited the summary of this revision. (Show Details)Wed, Mar 15, 1:12 PM
atul added a parent revision: D7081: [web] Update `getSignedIdentityKeysBlobSelector` to `await initOlm();`.
atul updated this revision to Diff 23770.Wed, Mar 15, 1:16 PM

rebase

Harbormaster completed remote builds in B17353: Diff 23770.Wed, Mar 15, 1:31 PM
atul added a child revision: D7082: [keyserver] Handle `SIGNED_IDENTITY_KEYS_BLOB` response.Wed, Mar 15, 1:42 PM
atul updated this revision to Diff 23775.Wed, Mar 15, 4:12 PM

rebase and land

This revision was landed with ongoing or failed builds.Wed, Mar 15, 4:20 PM
Closed by commit rCOMM6bfaf0746b8f: [web] Add `CryptoStore` to `persistConfig.whitelist` (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM6bfaf0746b8f: [web] Add `CryptoStore` to `persistConfig.whitelist`.
Harbormaster completed remote builds in B17357: Diff 23775.Wed, Mar 15, 4:27 PM

Revision Contents
Changeset List

PathSize
web/
2 lines

Diff 23776

View Options

web/root.js

Loading...