Skip to content

Migration Verification Script: Check Paused Methods#299

Merged
kiseln merged 7 commits intomigrationfrom
migration-verification-script
Mar 23, 2025
Merged

Migration Verification Script: Check Paused Methods#299
kiseln merged 7 commits intomigrationfrom
migration-verification-script

Conversation

@r-near
Copy link
Copy Markdown
Contributor

@r-near r-near commented Mar 16, 2025

This PR adds a script that verifies all contracts have the necessary methods paused before proceeding with migration.

Features

  • Checks both Ethereum and NEAR contracts
  • Handles transparent proxy contracts properly
  • Provides clear, color-coded output
  • Includes a migration checklist

How to Use

  1. pnpm install
  2. Use the consolidated .env file (no need for separate configs)
  3. pnpm start

The script is part of the verification toolkit and will be used to ensure safe migration of tokens between chains.

https://github.com/Near-One/omni-bridge-private/issues/35

Comment thread migration/verification/.env
@kiseln
Copy link
Copy Markdown
Contributor

kiseln commented Mar 20, 2025

And let's add production .env file @r-near

@r-near
Copy link
Copy Markdown
Contributor Author

r-near commented Mar 20, 2025

And let's add production .env file @r-near

Added!

Comment thread migration/.env.mainnet Outdated
Comment thread migration/.env.mainnet Outdated
Comment thread migration/.env.mainnet Outdated
Comment thread migration/.env.mainnet

# Token Locker
TOKEN_LOCKER=ft-locker.bridge.near
TOKEN_LOCKER_ADMIN=ft-locker.bridge.near
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This value doesn't seem correct

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I got it from here: https://github.com/Near-One/bridge-ops

Not sure what the right one is - we already have something for the Omni Bridge. I thought the Token Locker was slightly different?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

But why is FT_LOCKED_ADMIN same address?

@kiseln kiseln merged commit 9b64936 into migration Mar 23, 2025
4 of 11 checks passed
Comment thread migration/verification/verify-paused-methods.ts
Comment thread migration/verification/verify-paused-methods.ts
Comment thread migration/verification/verify-paused-methods.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants