This is the last step of the migration - we should no longer use the values returned from the initial Redux state.
Depends on D12153
https://linear.app/comm/issue/ENG-3486/move-entrystore-to-sqlite
Details
Details
Closed and opened the app (both native and web) - the entries aren't sent from the keyserver so doing this verified if they are correctly persisted and rehydrated.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| keyserver/src/responders/redux-state-responders.js | ||
|---|---|---|
| 223 ↗ | (On Diff #40527) | We still should return this because we don't persist it on web |
| keyserver/src/responders/redux-state-responders.js | ||
|---|---|---|
| 223 ↗ | (On Diff #40527) | If it's always 0, is it something we can just insert automatically on the web side? |
| keyserver/src/responders/redux-state-responders.js | ||
|---|---|---|
| 223 ↗ | (On Diff #40527) | Yes, that makes sense! |
| keyserver/src/responders/redux-state-responders.js | ||
|---|---|---|
| 223 ↗ | (On Diff #40527) | We insert this value on the web automatically by having it in the defaultWebState. Here we still should return this value so that the finalEntryStorePromise returns Promise<EntryStore>. |