Page MenuHomePhabricator

[native/web] introduce migration to move local message info to sqlite
ClosedPublic

Authored by ginsu on Sun, Jun 9, 11:02 PM.
Tags
None
Referenced Files
F2026741: D12370.id41246.diff
Mon, Jun 17, 5:47 PM
F2026434: D12370.id41229.diff
Mon, Jun 17, 4:10 PM
F2026429: D12370.id41230.diff
Mon, Jun 17, 4:04 PM
F2025790: D12370.id41249.diff
Mon, Jun 17, 11:59 AM
Unknown Object (File)
Sat, Jun 15, 5:47 PM
Unknown Object (File)
Sat, Jun 15, 3:54 PM
Unknown Object (File)
Sat, Jun 15, 3:53 PM
Unknown Object (File)
Sat, Jun 15, 3:35 PM
Subscribers

Details

Summary

This migration moves + saves any existing local message infos into sqlite

Depends on D12358

Test Plan

Created some failed sent local messages, ran migrations and confirmed that the there are db rows in the local message client db for each failed message. Double checked in redux dev tools that the migrations ran successfully and additionally assertMessageStoreLocalMessageInfosAreEqual (which gets introduced in the next diff in this stack) does not throw errors whenever I open the app

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu retitled this revision from [native/web] introduce migration to move local message info to sqlite to DRAFT [native/web] introduce migration to move local message info to sqlite.Sun, Jun 9, 11:09 PM
ginsu edited the summary of this revision. (Show Details)
ginsu added a subscriber: will.
ginsu requested review of this revision.Sun, Jun 9, 11:19 PM
ginsu retitled this revision from DRAFT [native/web] introduce migration to move local message info to sqlite to [native/web] introduce migration to move local message info to sqlite.Tue, Jun 11, 10:40 PM
ginsu edited the summary of this revision. (Show Details)
ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: kamil, will.
ginsu edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Wed, Jun 12, 4:11 AM