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

xtokens config #331

Merged
merged 24 commits into from
Oct 30, 2024
Merged

xtokens config #331

merged 24 commits into from
Oct 30, 2024

Conversation

ozgunozerk
Copy link
Collaborator

Configures orml_xtokens.

To do that, it also adds/configures the following pallets:

  • pallet_erc20_xcm_bridge
  • pallet_xcm_weight_trader

Quite a lot of types and some new logic is added. Followed mainly moonbeam's approach for the XCM strategies, but ofc changed/simplified some stuff. For example: we are not using pallet_collectives OpenTechCommitteeInstance or overnance::custom_origins::FastGeneralAdmin for submitting new assets to or deleting existing ones from the supported asset list. I left that decision to the end-user of the template with an inline comment.

@ozgunozerk ozgunozerk requested a review from KitHat October 14, 2024 06:47
@ozgunozerk ozgunozerk changed the title Ozgun xtokens config xtokens config Oct 14, 2024
@ozgunozerk ozgunozerk self-assigned this Oct 18, 2024
@ozgunozerk ozgunozerk linked an issue Oct 18, 2024 that may be closed by this pull request
3 tasks
Base automatically changed from ozgun-asset-hub-as-reserve to main October 21, 2024 04:59
Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for docs-oz-polkadot ready!

Name Link
🔨 Latest commit be1e2e0
🔍 Latest deploy log https://app.netlify.com/sites/docs-oz-polkadot/deploys/6721cf172299850008a82035
😎 Deploy Preview https://deploy-preview-331--docs-oz-polkadot.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ozgunozerk ozgunozerk force-pushed the ozgun-xtokens-config branch from a64764c to cfbc952 Compare October 21, 2024 05:16
@ozgunozerk ozgunozerk requested a review from 4meta5 October 24, 2024 12:29
Copy link
Contributor

@4meta5 4meta5 left a comment

Choose a reason for hiding this comment

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

great this contains some types needed for #263

@4meta5 4meta5 mentioned this pull request Oct 28, 2024
11 tasks
@ozgunozerk ozgunozerk merged commit c43aa95 into main Oct 30, 2024
4 of 6 checks passed
@ozgunozerk ozgunozerk deleted the ozgun-xtokens-config branch October 30, 2024 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

🎁 [Feature Request]: Configure XTokens for EVM template 🎁 [Feature Request]: Snowbridge Integration
4 participants