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

Unify components for Polkadot bridges #7274

Open
6 tasks
bkontur opened this issue Jan 21, 2025 · 0 comments
Open
6 tasks

Unify components for Polkadot bridges #7274

bkontur opened this issue Jan 21, 2025 · 0 comments

Comments

@bkontur
Copy link
Contributor

bkontur commented Jan 21, 2025

Description

At least P/K and Snowbridge shares some commonalities (xcm routing, congestion, dynamic fees, collecting rewards, ...) which are unnecessary implemented twice.

TBD:

Trait changes:

  • ExporterFor change Option<Asset> to Assets or Option<Assets to support more fees and be aligned with SendXcm trait
  • ExportXcm - remove channel and change the order or params:
    fn validate(
     	universal_source: &mut Option<InteriorLocation>,
     	network: NetworkId,
     	destination: &mut Option<InteriorLocation>,
     	message: &mut Option<Xcm<()>>,
    ) -> SendResult<Self::Ticket>;
    
@bkontur bkontur converted this from a draft issue Jan 21, 2025
@acatangiu acatangiu changed the title Simplify/unify bridges Unify components for Polkadot bridges Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

1 participant