Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Moonriver] [Moonbeam] Add missing EvmForeignAssets transactor #3068

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

Agusrodri
Copy link
Contributor

@Agusrodri Agusrodri commented Dec 2, 2024

What does it do?

Fixes the XcmExecutorConfig for Moonriver and Moonbeam runtimes, adding the missing EvmForeignAssets transactor for handling evm foreign asset deposits/withdrawals when receiving these kind of assets via XCM.

@Agusrodri Agusrodri added B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit breaking Needs to be mentioned in breaking changes labels Dec 2, 2024
Copy link
Contributor

github-actions bot commented Dec 2, 2024

WASM runtime size check:

Compared to target branch

Moonbase runtime: 2248 KB (no changes) ✅

Moonbeam runtime: 2228 KB (no changes) ✅

Moonriver runtime: 2232 KB (no changes) ✅

Compared to latest release (runtime-3300)

Moonbase runtime: 2248 KB (+220 KB compared to latest release) ⚠️

Moonbeam runtime: 2228 KB (+232 KB compared to latest release) ⚠️

Moonriver runtime: 2232 KB (+240 KB compared to latest release) ⚠️

Copy link
Contributor

github-actions bot commented Dec 3, 2024

Coverage Report

@@                    Coverage Diff                     @@
##           master   agustin-fix-xcm-configs     +/-   ##
==========================================================
  Coverage   74.70%                    74.70%   0.00%     
  Files         369                       369             
  Lines       94308                     94308             
==========================================================
+ Hits        70445                     70446      +1     
- Misses      23863                     23862      -1     
Files Changed Coverage
/pallets/moonbeam-foreign-assets/src/lib.rs 70.85% (+0.45%) 🔼

Coverage generated Tue Dec 3 00:13:37 UTC 2024

@crystalin crystalin merged commit c69c0cb into master Dec 3, 2024
47 checks passed
@crystalin crystalin deleted the agustin-fix-xcm-configs branch December 3, 2024 07:11
@RomarQ RomarQ removed the breaking Needs to be mentioned in breaking changes label Dec 4, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit evm-native-foreign-assets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants