Page MenuHomePhabricator

[lib/native] lift createUpdateDBOpsForThreadStoreThreadInfos into lib
ClosedPublic

Authored by ginsu on Jun 19 2024, 5:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 1, 1:57 PM
Unknown Object (File)
Oct 12 2024, 1:00 PM
Unknown Object (File)
Oct 12 2024, 1:00 PM
Unknown Object (File)
Oct 12 2024, 1:00 PM
Unknown Object (File)
Oct 12 2024, 12:58 PM
Unknown Object (File)
Oct 12 2024, 12:58 PM
Unknown Object (File)
Sep 12 2024, 10:21 AM
Unknown Object (File)
Sep 12 2024, 10:21 AM
Subscribers

Details

Summary

With migration 77, I had a huge misunderstanding about the new migration approach. I thought for this migration we needed to create and directly return a new thread store state to redux using .processStoreOperations(). However, we actually should only just return the operations and just return state as is. The threadStore will be read by SQLiteDataHandler which populates redux with the migrated data from the ops.

Anyways, we need this function for both native + web migrations, this diff lifts this function into lib

Depends on D12497

Test Plan

flow + at the end of the stack confirmed that migration 78 works as expected

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: ashoat, kamil.
ginsu requested review of this revision.Jun 19 2024, 6:42 PM
This revision is now accepted and ready to land.Jun 19 2024, 6:55 PM