[web] Replace persisted entries with the ones from initial redux state
ClosedPublic
Actions

Authored by tomek on May 22 2024, 4:34 AM.

Details

Reviewers

kamil
marcin

Commits

rCOMM70681d12e108: [web] Replace persisted entries with the ones from initial redux state

Summary

After we conclude that the migration was successful, we will replace this code with the one that replaces the entries only when they are missing from the store.

Depends on D12164

https://linear.app/comm/issue/ENG-3486/move-entrystore-to-sqlite

Test Plan

Run console.log(entryStoreOperations) from tis code.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.May 22 2024, 4:34 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMay 22 2024, 4:34 AM

tomek added a child revision: D12153: [native] Persist only lastUserInteractionCalendar from the entry store.May 22 2024, 4:34 AM

Harbormaster completed remote builds in B29154: Diff 40525.May 22 2024, 4:50 AM

tomek requested review of this revision.May 22 2024, 4:50 AM

marcin accepted this revision.May 22 2024, 7:14 AM

This revision is now accepted and ready to land.May 22 2024, 7:14 AM

inka added a subscriber: inka.May 24 2024, 9:01 AM

inka added inline comments.

web/redux/initial-state-gate.js
160–162 ↗	(On Diff #40525)	We should probably also check if `entryStoreOperations.length > 0`

marcin edited the test plan for this revision. (Show Details)May 24 2024, 9:14 AM

ashoat added inline comments.May 25 2024, 7:21 AM

web/redux/initial-state-gate.js
160–162 ↗	(On Diff #40525)	Good catch – we definitely need to check that here

Check if there are some entry store ops

Harbormaster completed remote builds in B29251: Diff 40657.May 27 2024, 4:44 AM

Closed by commit rCOMM70681d12e108: [web] Replace persisted entries with the ones from initial redux state (authored by tomek). · Explain WhyMay 27 2024, 5:51 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM70681d12e108: [web] Replace persisted entries with the ones from initial redux state.

Revision Contents
Changeset List

Path

Size

web/

redux/

initial-state-gate.js

13 lines

Diff 40669

View Options

web/redux/initial-state-gate.js

[web] Replace persisted entries with the ones from initial redux stateClosedPublicActions