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 Neon Devnet deployments #214

Merged
merged 3 commits into from
Mar 29, 2023
Merged

Add Neon Devnet deployments #214

merged 3 commits into from
Mar 29, 2023

Conversation

Sj-001
Copy link
Contributor

@Sj-001 Sj-001 commented Mar 26, 2023

@moisses89
Copy link
Member

@Sj-001 must be necessary to create an issue to add new-chain and provide all the required information to review this PR.

@Sj-001
Copy link
Contributor Author

Sj-001 commented Mar 27, 2023

Hey @moisses89 , I tried to create the issue but I keep getting 500 server error.

1 similar comment
@Sj-001
Copy link
Contributor Author

Sj-001 commented Mar 27, 2023

Hey @moisses89 , I tried to create the issue but I keep getting 500 server error.

@moisses89
Copy link
Member

@Sj-001
Copy link
Contributor Author

Sj-001 commented Mar 27, 2023

Hey @moisses89 , I tried to create the issue but I keep getting 500 server error.

Here https://github.com/safe-global/safe-deployments/issues/new?assignees=&labels=%5Benhancement%5D&template=new_chain.yaml&title=%5BNew+chain%5D%3A+

Created. please check @moisses89

Copy link
Member

@moisses89 moisses89 left a comment

Choose a reason for hiding this comment

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

We only accept deterministic addresses here.
To do that you should use the singleton factory to deploy the contracts, be careful to use the correct branch https://github.com/safe-global/safe-contracts#deploy
But first singleton factory should be deployed safe-global/safe-singleton-factory#112

@@ -95,6 +95,7 @@
"421611": "0xf48f2B2d2a534e402487b3ee7C18c33Aec0Fe5e4",
"421613": "0xf48f2B2d2a534e402487b3ee7C18c33Aec0Fe5e4",
"11155111": "0x017062a1dE2FE6b99BE3d9d37841FeD19F573804",
"245022926": "0xAb5209Fbd1512008836259Cdb317199ED57faA83",
Copy link
Member

Choose a reason for hiding this comment

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

This address is not the deterministic address

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hey @moisses89 , I deployed the safe-contracts again using 0x914d7fec6aac8cd542e72bca78b30650d45643d7 as the singleton factory deployed by @Uxio0 on Neon. Please review again.

Copy link
Member

Choose a reason for hiding this comment

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

@Sj-001 which tag branch did you use on safe-contracts? you should use v1.3.0-libs.0 as is described here https://github.com/safe-global/safe-contracts#deploy
I'm asking because are not still the deterministic addresses.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gotcha, deployed again, please review, I think its the correct one now:) @moisses89

@Sj-001 Sj-001 requested review from moisses89 and removed request for rmeissner, Uxio0, mmv08 and hectorgomezv March 28, 2023 13:50
Copy link
Member

@moisses89 moisses89 left a comment

Choose a reason for hiding this comment

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

Looks good to me! 🚀

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.

[New chain]:Neon EVM DevNet
2 participants