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

Add Fantom mainnet and testnet #6

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jenikd
Copy link

@jenikd jenikd commented Jun 14, 2023

Adds Safe allowance module deployments for Fantom testnet and mainnet.

Address is different as this contracts were deployed before a mechanism of same addresses on all chains.

Block explorer: ftmscan.com

Copy link

@MissLov3ly MissLov3ly left a comment

Choose a reason for hiding this comment

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

=) THANK YOU DEAR FRIEND!****

@jenikd
Copy link
Author

jenikd commented Jan 30, 2024

Hi @MissLov3ly, because of your time needed for review, the main branch always have another updates which change same file as this PR. So I have to rebase this branch again to the main branch to be able to merge. But after this, it again needs review as nothing more changed. And we are in kind of circle with this.
We have to align somehow with this, or you can merge it by yourself after doing review.

Copy link
Collaborator

@nlordell nlordell left a comment

Choose a reason for hiding this comment

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

Because of deterministic deployments, the addresses should end up on one of the two deterministic values: 0xCFbFaC74C26F8647cBDb8c5caf80BB5b32E43134 or 0x1Fb403834C911eB98d56E74F5182b0d64C3b3b4D.

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