Page MenuHomePhabricator

[lib] introduce updateThreadStoreThreadInfos
ClosedPublic

Authored by ginsu on Jun 12 2024, 3:11 AM.
Tags
None
Referenced Files
F3526211: D12400.diff
Mon, Dec 23, 9:11 PM
Unknown Object (File)
Sat, Dec 21, 9:29 AM
Unknown Object (File)
Thu, Dec 19, 9:44 AM
Unknown Object (File)
Thu, Dec 19, 9:44 AM
Unknown Object (File)
Thu, Dec 19, 9:44 AM
Unknown Object (File)
Thu, Dec 19, 9:44 AM
Unknown Object (File)
Thu, Dec 19, 9:44 AM
Unknown Object (File)
Tue, Dec 17, 5:28 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