-
Notifications
You must be signed in to change notification settings - Fork 4
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 bridge dispatcher and exporter #33
base: main
Are you sure you want to change the base?
Conversation
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!
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 am just curious, maybe I've missed that somewhere,
I see type MessageExporter = ToBridgeHubPolkadotHaulBlobExporter;
,
so just one question, what kind of calls will send this chain to the BridgeHub?
I have been said that they'll be calling some stuff on that new Polkadot parachain (referenced as |
@zdave-parity When you have a moment, please review and approve if it is ok - we need an approval to merge. Thanks |
The fully working bridge may be started by using branches, mentioned in paritytech/parity-bridges-common#2547. I'm going to open multiple PRs to make it easier to review - some PRs need a different set of reviewers
This PR adds message dispatcher for inbound messages and exporter for outbound messages. Two differences with our "regular" configuration:
XcmExecutor
, hence theWithXcmWeightDispatcher
.@acatangiu @bkontur @serban300 Please, take a look.