Paths

Table of Contentst

Differential D12014

[lib] Move createAsyncMigrate to utils
ClosedPublic
Actions

Authored by tomek on May 13 2024, 5:10 AM.

Tags

None

Referenced Files

	F3515104: D12014.id40949.diff
	Sun, Dec 22, 7:30 AM

	F3515077: D12014.id40931.diff
	Sun, Dec 22, 7:24 AM

	F3514756: D12014.id40228.diff
	Sun, Dec 22, 6:26 AM

	F3514751: D12014.id40109.diff
	Sun, Dec 22, 6:25 AM

	F3513678: D12014.diff
	Sun, Dec 22, 1:28 AM

	Unknown Object (File)
	Fri, Dec 20, 5:50 PM

	Unknown Object (File)
	Wed, Dec 4, 7:09 PM

	Unknown Object (File)
	Nov 11 2024, 8:43 PM

View All 94 Files

Subscribers

Details

Reviewers

kamil
marcin
varun

Commits

rCOMMc75e3a4b8a2a: [lib] Move createAsyncMigrate to utils

Summary

I need to split createAsyncMigrate function and after this refactoring it won't make sense for these to be in create-async-migrate.js - the new one should bo somewhere in utils. The original createAsyncMigrate should also be placed in utils probably from the beginning.

https://linear.app/comm/issue/ENG-7009/run-redux-migrations-after-the-backup-is-restored

Depends on D11947

Test Plan

Flow

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.May 13 2024, 5:10 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMay 13 2024, 5:10 AM

tomek added inline comments.May 13 2024, 5:12 AM

lib/utils/migration-utils.js
235 ↗	(On Diff #40109)	For some reason `jest` was failing on importing the `DEFAULT_VERSION` which was used here originally. I think we can safely use `-1` here because it is tightly coupled with our migrations versions and the only constraint is for it to be lower than the version of any of them.

Harbormaster completed remote builds in B28866: Diff 40109.May 13 2024, 5:26 AM

tomek requested review of this revision.May 13 2024, 5:26 AM

tomek added a child revision: D12015: [lib] Extract a function that runs the migrations.May 13 2024, 6:16 AM

kamil accepted this revision.May 14 2024, 11:45 PM

This revision is now accepted and ready to land.May 14 2024, 11:45 PM

Rebase

Harbormaster completed remote builds in B28944: Diff 40228.May 15 2024, 8:33 AM

tomek added a parent revision: D11947: [web] Check backup version when restoring.May 20 2024, 3:11 AM

tomek removed a parent revision: D12013: [native] Verify if the backup version is correct.

Rebase

tomek edited the summary of this revision. (Show Details)Jun 3 2024, 8:32 AM

tomek removed a parent revision: D12013: [native] Verify if the backup version is correct.

Harbormaster failed remote builds in B29383: Diff 40882!Jun 3 2024, 9:11 AM

Rebase

Harbormaster completed remote builds in B29430: Diff 40931.Jun 4 2024, 3:36 AM

Closed by commit rCOMMc75e3a4b8a2a: [lib] Move createAsyncMigrate to utils (authored by tomek). · Explain WhyJun 4 2024, 5:45 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMc75e3a4b8a2a: [lib] Move createAsyncMigrate to utils.

Revision Contents
Changeset List

Path

Size

lib/

shared/

create-async-migrate.js

utils/

migration-utils.js

122 lines

native/

redux/

2 lines

web/

redux/

6 lines

Diff 40949

lib/shared/create-async-migrate.js

Loading...

lib/utils/migration-utils.js

Loading...

native/redux/persist.js

Loading...

web/redux/persist.js

Loading...