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

feat: add xrpl-evm-devnet #553

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AdriaCarrera
Copy link

No description provided.

@AdriaCarrera AdriaCarrera requested a review from a team as a code owner February 12, 2025 17:29
@AdriaCarrera AdriaCarrera requested review from npty and ahramy February 12, 2025 17:29
"tokenSymbol": "XRP",
"confirmations": 1,
"finality": "finalized",
"approxFinalityWaitTime": 1,
Copy link
Contributor

Choose a reason for hiding this comment

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

@AdriaCarrera any docs on finality mechanism I can review?

"name": "Blockscout",
"url": "https://explorer.xrplevm.org",
"api": "https://explorer.xrplevm.org/api",
"apiKey": "empty"
Copy link
Contributor

Choose a reason for hiding this comment

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

You can remove "apiKey": "empty" from the config.

"address": "0x6850335DA5b6E960eB509B7ec4D159ebA8e89eFd",
"implementation": "0x35dE9395c6eC7F12dbb21f9947E24C3592b85A7F",
"implementationCodehash": "0x70edf88fd63576602e293de1d994e77541daf1a885e97800d9b6c94d7d8482ab",
"deploymentMethod": "create3",
Copy link
Contributor

Choose a reason for hiding this comment

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

Deployment method for AxelarGateway should be create2.

"deploymentMethod": "create3",
"previousSignersRetention": 15,
"domainSeparator": "0x8a1a34e4e527215bb55d1fa7996fafae63ca752ca9224cd7c0249b11b33554ff",
"minimumRotationDelay": 86400,
Copy link
Contributor

Choose a reason for hiding this comment

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

minimumRotationDelay on devnet should be 0.

"minimumRotationDelay": 86400,
"connectionType": "amplifier",
"owner": "0x9159C650e1D7E10a17c450eb3D50778aBA593D61",
"salt": "xrpl-evm-devnet"
Copy link
Contributor

Choose a reason for hiding this comment

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

It should be the release tag of AxelarGateway used, something like: v6.0.4

"owner": "0x9159C650e1D7E10a17c450eb3D50778aBA593D61",
"address": "0x217D3F23884beD1B13177DaC309634E4A30fe5F1",
"deployer": "0x9159C650e1D7E10a17c450eb3D50778aBA593D61",
"deploymentMethod": "create3",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be create2

},
"InterchainTokenFactory": {
"deployer": "0x9159C650e1D7E10a17c450eb3D50778aBA593D61",
"salt": "ITS Factory xrpl-evm-devnet devnet-amplifier v0",
Copy link
Contributor

Choose a reason for hiding this comment

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

I see salts included in the contract deployments where they are not present for other amplifier chains like flow, as well as the salt is different from the defaults. would like to discuss the reasoning behind it.

Comment on lines +1059 to +1060
"1",
"1"
Copy link
Contributor

Choose a reason for hiding this comment

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

The value is 6/10 for devnet-amplifier.

Comment on lines +1205 to +1206
"1",
"1"
Copy link
Contributor

Choose a reason for hiding this comment

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

6/10 here as hell.

@@ -1089,6 +1198,21 @@
"domainSeparator": "0x293407cef6cb084c2c7ca31794d4cc95d8c3ea2aced086aebb518a2c37b28dc6",
"address": "axelar1uuja08qdy790ewq9mz0r6t7c4yp9fqccu9f9zssr4tu4k0l52yeqjhckm3"
},
"xrpl-evm-devnet": {
"governanceAddress": "axelar1zlr7e5qf3sz7yf890rkh9tcnu87234k6k7ytd9",
"adminAddress": "axelar1hxcz4z263uxvlsmrj2pzfhmy8ggwwqjwu58w4t",
Copy link
Contributor

Choose a reason for hiding this comment

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

axelar1zlr7e5qf3sz7yf890rkh9tcnu87234k6k7ytd9 is used as the admin address on devnet-amplifier.

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.

2 participants