Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

fix: change the ToposCore admin to subnet validator #100

Closed
wants to merge 1 commit into from

Conversation

JDawg287
Copy link
Member

@JDawg287 JDawg287 commented Aug 11, 2023

Description

This PR modifies the topos messaging protocol deployment script to change the admin of the ToposCore contract from the static deployer to the subnet validator.

Additions and Changes

  • Added logic to transfer some funds from the deployer account to the sequencer account
  • Changed the toposCoreProxyParams to the validator key in deploy-topos-msg-protocol.ts script
  • Changed the toposCoreProxyParams to the validator key in deploy-topos-msg-protocol-dynamic.ts script
  • Changed deployer's wallet to sequencer's wallet when setting up the subnetId as the sequencer would now be the owner of the ToposCore contract

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@JDawg287 JDawg287 self-assigned this Aug 11, 2023
@JDawg287
Copy link
Member Author

Closing this in favor or #98

@JDawg287 JDawg287 closed this Aug 16, 2023
@JDawg287 JDawg287 deleted the fix/change-topos-core-admin branch August 16, 2023 22:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant