-
Notifications
You must be signed in to change notification settings - Fork 75
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
Connectors v0.1.0 #1160
Connectors v0.1.0 #1160
Conversation
76e4ff6
to
1a3f64b
Compare
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.
Some notes for the reviewers
Domain only needs ConnectorEncode and let's drop Decode since it's not needed for now.
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.
LGTM! I left some minor comments
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.
LGTM! I have a few minor comments
Co-authored-by: William Freudenberger <[email protected]>
Thanks William
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.
LGTM!
Connectors v1
The latest revision of this PR has been tested on the Moonbase Alpha testnet and has passed all the tests where all the connectors messages (
AddPool
,AddTranche
,UpdateMember
,UpdateTokenPrice
, andTransfer
) were successfully sent from the Centrifuge development runtime through XCM to Moonbase, where they were forwarded throughethereum_xcm
to the Moonbase EVM down to ourConnectorXcmRouter
smart contract.The changes which close the v1 are:
Domain
andDomainAddress
types to best map the mental model we have for connectorsprice
field to theAddTranche
message with the latest tranche priceTo Do
initial_price
toMessage::AddTranche
(Connectors v1 liquidity-pools#16 (comment))gas_limit
to max or have user pass it down when submitting a callupdate_member
derivation of DomainAddress