HomePhabricator
Diffusion Comm 32270fba0afa

[lib] remove updateThreadStoreThreadInfos util function

Description

[lib] remove updateThreadStoreThreadInfos util function

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, this util function is unused and can be removed

Depends on D12498

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

Reviewers: ashoat, kamil

Reviewed By: ashoat

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D12499

Details

Provenance
ginsuAuthored on Jun 19 2024, 4:48 PM
Reviewer
ashoat
Differential Revision
D12499: [lib] remove updateThreadStoreThreadInfos util function
Parents
rCOMM0462bf7cc9b0: [lib/native] lift createUpdateDBOpsForThreadStoreThreadInfos into lib
Branches
Unknown
Tags
Unknown