Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider TeleporterRegistry deployment through Nick's method #76

Closed
minghinmatthewlam opened this issue Oct 26, 2023 · 1 comment
Closed
Labels
enhancement New feature or request upgradeability

Comments

@minghinmatthewlam
Copy link
Contributor

Context and scope
TeleporterRegistry is currently implemented without Nick's method, so will end up at different addresses across different subnet chains. The registry needs Warp out-of-band messages to update, and do so validators need to add the Warp out-of-band message bytes, and we want to streamline this process as much as possible.

Discussion and alternatives
We would need to remove the calls in the registry' constructor, which include registry initialization and getting warp blockchain ID, out of constructor, to prevent the deployment from failing and using up the nonce.

Open questions
The destination chain ID between Warp messages would be different already anyways, how can we streamline this too?

@michaelkaplan13
Copy link
Collaborator

Closing this issue since the deploy_registry.sh script handles the deployment of the registry, which doesn't need to be at the same address on every chain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request upgradeability
Projects
Archived in project
Status: ✍️ Design
Development

No branches or pull requests

2 participants