Skip to content

Conversation

@mcmire
Copy link
Contributor

@mcmire mcmire commented May 13, 2025

Description

Migration files should never be removed once added, as this creates a mismatch between the the number of the final migration and the total number of migrations, and so new migrations that are added later could be skipped.

This commit adds back migration 73, which was removed in a previous commit, except that it replaces it with a version that does nothing.

Related issues

Fixes #15306.

Manual testing steps

  • Check out this branch.
  • Open app/store/migrations/index.ts and comment out the final migration (78).
  • Open app/store/persistConfig.ts and change migrate: createMigrate(migrations, { debug: false }), to migrate: createMigrate(migrations, { debug: true }),.
  • Run yarn setup:expo and yarn watch:clean.
  • Remove the app from your device or local emulator first (you can also simply clear all app storage on Android).
  • Open the app, go through onboarding, and create an account.
  • Now reopen app/store/migrations/index.ts and uncomment migration 78 so that it will run.
  • Reload the app.
  • Look for a line starting with redux-persist. You should see that migration 78 ran (not 77).

Screenshots/Recordings

Before

After

This is what you should see:

Screenshot 2025-05-13 at 3 20 44 PM

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Migration files should never be removed once added, as this creates a
mismatch between the the number of the final migration and the total
number of migrations (which needs to match).
@github-actions
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-wallet-framework-deprecated DEPRECATED: please use "team-core-platform" instead label May 13, 2025
@mcmire mcmire marked this pull request as ready for review May 13, 2025 21:21
@mcmire mcmire requested a review from a team May 13, 2025 21:21
@mcmire mcmire changed the title fix: cp-7.46.0 cp-7.47.0 Add dummy migration for previously removed 73 fix: cp-7.46.0 cp-7.47.0 Add dummy migration 73 May 13, 2025
@mcmire mcmire added the No QA Needed Apply this label when your PR does not need any QA effort. label May 13, 2025
@mcmire mcmire changed the title fix: cp-7.46.0 cp-7.47.0 Add dummy migration 73 fix: cp-7.46.0 cp-7.47.0 Add dummy file for migration 73 May 13, 2025
Copy link
Contributor

@Cal-L Cal-L left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@mcmire mcmire enabled auto-merge May 13, 2025 22:22
@github-actions
Copy link
Contributor

github-actions bot commented May 14, 2025

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 6daa203
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/115638ab-d7d1-4382-a1a8-c1f1a181568c

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@sonarqubecloud
Copy link

@NicolasMassart NicolasMassart added No QA Needed Apply this label when your PR does not need any QA effort. No E2E Smoke Needed and removed No QA Needed Apply this label when your PR does not need any QA effort. Run Smoke E2E labels May 14, 2025
@mcmire mcmire added this pull request to the merge queue May 14, 2025
Merged via the queue into main with commit 230571f May 14, 2025
52 of 54 checks passed
@mcmire mcmire deleted the add-dummy-migration-for-73 branch May 14, 2025 12:16
@github-actions github-actions bot locked and limited conversation to collaborators May 14, 2025
@metamaskbot metamaskbot added the release-7.48.0 Issue or pull request that will be included in release 7.48.0 label May 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

No QA Needed Apply this label when your PR does not need any QA effort. release-7.48.0 Issue or pull request that will be included in release 7.48.0 team-wallet-framework-deprecated DEPRECATED: please use "team-core-platform" instead

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migration 73 is missing

6 participants