Skip to content

Conversation

@zjb0807
Copy link
Contributor

@zjb0807 zjb0807 commented Sep 24, 2025

  • add XtokensParameters to store ReserveLocation with orml_parameters pallet
  • add ParametersMigrateToV1 to setup ReserveLocation of XtokensParameters

Since the chain_part and non_chain_part called by xtokens are hard-coded, if we want to use xtokens after AHM, we need to upgrade the runtime first.

@zjb0807 zjb0807 requested a review from xlc September 24, 2025 00:04
@codecov
Copy link

codecov bot commented Sep 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.06%. Comparing base (069ed19) to head (487d124).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2915      +/-   ##
==========================================
- Coverage   61.12%   61.06%   -0.07%     
==========================================
  Files          69       70       +1     
  Lines        9449     9554     +105     
==========================================
+ Hits         5776     5834      +58     
- Misses       3673     3720      +47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@zjb0807
Copy link
Contributor Author

zjb0807 commented Sep 24, 2025

It seems that adding MigratedStatus storage to xtokens to control it would be better. No runtime upgrade

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants