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.

Sign up for GitHub

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

Jump to bottom

fix: revert deploy changes #25

Closed
wants to merge 2 commits into from
Closed

fix: revert deploy changes #25

wants to merge 2 commits into from

Conversation

JDawg287
Copy link
Member

@JDawg287 JDawg287 commented Sep 7, 2023

Description

Revert the changes made by #23 in favour of using constant addresses for deploying smart contracts, for ease of redeployment.

Fixes TP-700

Additions and Changes

PR Checklist:

  • I have performed a self-review of my code
  • I have commented on 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 Sep 7, 2023
@JDawg287 JDawg287 changed the title Fix/revert deploy changes fix/revert-deploy-changes Sep 7, 2023
@JDawg287 JDawg287 changed the title fix/revert-deploy-changes fix: revert deploy changes Sep 7, 2023
@JDawg287
Copy link
Member Author

JDawg287 commented Sep 7, 2023

Closing as this is a duplicate of changes already on main

@JDawg287 JDawg287 closed this Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sebastiendan sebastiendan Awaiting requested review from sebastiendan

Assignees

@JDawg287 JDawg287

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JDawg287