-
Notifications
You must be signed in to change notification settings - Fork 172
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
5834 x1 for mainnet - Main #6130
Conversation
…repo into 5834-x1-for-mainnet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need the graph client built to the latest state of the branch ?
LGTM otherwise
I dont think so, as the latest state of |
@@ -214,9 +215,10 @@ export const XERC20REGISTRY_DOMAIN_ADDRESS: Record<string, string> = { | |||
"1650553709": "0xcc1ce162fb1d70a377d1542873b1db57f5ff14d1", // base | |||
"1836016741": "0x689cCB572262e0ca66146FE5Ed5D998543d3AA3B", // mode | |||
"1935897199": "0x397aEEEDd44f40326f9eB583a1DFB8A7A673C40B", // scroll | |||
"2020368761": "", // xlayer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Registry + adapter to be deployed in separate PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, Will add it separately in sdk
Description
Feat: Adding x Layer
Type of change
High-level change(s) description - from the user's perspective
Related Issue(s)
Fixes
Related pull request(s)