Page MenuHomePhabricator

[native] Move MessageStoreTransform to lib
ClosedPublic

Authored by michal on Apr 3 2024, 6:07 AM.
Tags
None
Referenced Files
F3489786: D11537.diff
Wed, Dec 18, 1:37 PM
Unknown Object (File)
Nov 17 2024, 8:29 AM
Unknown Object (File)
Nov 3 2024, 2:09 AM
Unknown Object (File)
Oct 11 2024, 12:18 PM
Unknown Object (File)
Oct 11 2024, 12:18 PM
Unknown Object (File)
Oct 11 2024, 12:18 PM
Unknown Object (File)
Oct 11 2024, 12:17 PM
Unknown Object (File)
Oct 11 2024, 12:17 PM
Subscribers

Details

Summary

ENG-3490 : Migrate stores to SQLite on web
Now that we have will start persisting message store in sqlite on web, we need to have a redux transform, like on native which will split data between persited by redux and persisted by sqlite. This diff moves the transform to lib (+ updated comments). The transformer will be added to web in the later diffs.

Depends on D11536

Test Plan

Check if the app on native works. Check if rehydrated store contains data from both sources.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

lib/shared/transforms/message-store-transform.js
12–13

There are some changes to the comments, but no logic changes.

michal requested review of this revision.Apr 3 2024, 6:22 AM
kamil added inline comments.
lib/shared/transforms/message-store-transform.js
2

add empty line here

This revision is now accepted and ready to land.Apr 4 2024, 9:06 AM