Page MenuHomePhabricator

[native] Move MessageStoreTransform to lib
ClosedPublic

Authored by michal on Wed, Apr 3, 6:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 5:53 PM
Unknown Object (File)
Fri, Apr 12, 2:15 AM
Unknown Object (File)
Thu, Apr 11, 11:24 AM
Unknown Object (File)
Wed, Apr 10, 5:18 PM
Unknown Object (File)
Wed, Apr 10, 4:14 PM
Unknown Object (File)
Wed, Apr 10, 12:56 PM
Unknown Object (File)
Wed, Apr 10, 9:36 AM
Unknown Object (File)
Wed, Apr 10, 6:57 AM
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
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lib/shared/transforms/message-store-transform.js
12–13 ↗(On Diff #38724)

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

michal requested review of this revision.Wed, Apr 3, 6:22 AM
kamil added inline comments.
lib/shared/transforms/message-store-transform.js
2 ↗(On Diff #38724)

add empty line here

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