Page MenuHomePhabricator

[lib] introduce updateThreadStoreThreadInfos
ClosedPublic

Authored by ginsu on Jun 12 2024, 3:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 6, 2:39 AM
Unknown Object (File)
Tue, Sep 3, 8:35 PM
Unknown Object (File)
Tue, Sep 3, 8:35 PM
Unknown Object (File)
Tue, Sep 3, 8:35 PM
Unknown Object (File)
Tue, Sep 3, 8:35 PM
Unknown Object (File)
Tue, Sep 3, 8:35 PM
Unknown Object (File)
Aug 25 2024, 9:14 PM
Unknown Object (File)
Aug 3 2024, 4:27 PM
Subscribers

Details

Summary

This is the util function that will be called in both web and native persist.js files to update thread infos in the thread store using the new migration approach.

To implement this method I used the preexisting createUpdateDBOpsForThreadStoreThreadInfos util function as inspiration.

Test Plan

Tested further in the stack and will flesh out test plan there, but at the end of the stack tested new clients, old clients, and the state sync mechanism and everything worked as expected

As I was testing, I logged out the value of stateCheckStatus in handleResponsesClientSocketMessage and kept getting the following log:

stateCheckStatus { status: 'state_validated' }

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: tomek, kamil.
ginsu edited the summary of this revision. (Show Details)
ginsu edited the summary of this revision. (Show Details)
ginsu requested review of this revision.Jun 12 2024, 3:27 AM
This revision is now accepted and ready to land.Jun 12 2024, 4:13 AM