[lib][web][native] migrate backup data from non-backup tables
ClosedPublic
Actions

Authored by kamil on Jun 23 2025, 2:30 PM.

Details

Reviewers

tomek
bartek

Commits

rCOMMa1f65d3b236b: [lib][web][native] migrate backup data from non-backup tables

Summary

ENG-9784.

Depends on D14865

Test Plan

Checkout code before this stack.
Create some thick thread data (threads, entries, messages)
Patch this code, build app and trigger backup.
Login secondary device
Check that data create before code in this stack existed is included in backup

Diff Detail

Repository

rCOMM Comm

Branch

publish-2

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Jun 23 2025, 2:30 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJun 23 2025, 2:30 PM

Harbormaster completed remote builds in B34446: Diff 48540.Jun 23 2025, 3:25 PM

This wasn't tested yet (in progress) but publishing for review ahead of time. I am going to test it together with the Final testing task, which is time time-consuming process

tomek accepted this revision.Jun 24 2025, 9:19 AM

tomek added inline comments.

lib/utils/migrate-backup-data.js
125
133–140	It is a good idea to add some comments here
143

This revision is now accepted and ready to land.Jun 24 2025, 9:19 AM

address review

In D14866#410390, @kamil wrote:

This wasn't tested yet (in progress), but publishing for review ahead of time. I am going to test it together with the Final testing task, which is time time-consuming process

Tested, I described a detailed test plan in ENG-9681

Harbormaster completed remote builds in B34457: Diff 48564.Jun 24 2025, 10:56 AM

Closed by commit rCOMMa1f65d3b236b: [lib][web][native] migrate backup data from non-backup tables. · Explain WhyJun 24 2025, 11:09 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMa1f65d3b236b: [lib][web][native] migrate backup data from non-backup tables.