optimism icon indicating copy to clipboard operation
optimism copied to clipboard

op-chain-ops: sent message migration-data parsing

Open tynes opened this issue 3 years ago • 4 comments

Description

Add go code for reading output of the migration-data package.

tynes avatar Oct 24 '22 20:10 tynes

⚠️ No Changeset found

Latest commit: 4cf68cbdfd38f58e1b66210dd3880d0975ee2ad7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Oct 24 '22 20:10 changeset-bot[bot]

This PR changes implementation code, but doesn't include a changeset. Did you forget to add one?

mergify[bot] avatar Oct 24 '22 20:10 mergify[bot]

@tynes is this ready for review? Saw a wip commit. Happy to review if it is.

mslipper avatar Oct 26 '22 04:10 mslipper

@tynes is this ready for review? Saw a wip commit. Happy to review if it is.

Just converted it to a draft, sorry about that - I pushed some additional code to it and need to double check on the validation logic before its really ready for review.

Planning on a different PR that modularizes the balance migration stuff so that it could be used as part of this script, or we could just leave the balance migration stuff as is and run 2 scripts to do the full migration

tynes avatar Oct 26 '22 05:10 tynes

This PR has been added to the merge queue, and will be merged soon.

mergify[bot] avatar Oct 26 '22 19:10 mergify[bot]

This PR is next in line to be merged, and will be merged as soon as checks pass.

mergify[bot] avatar Oct 26 '22 19:10 mergify[bot]