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

xcm: avoid subtracting delivery fees twice #7201

Open
wants to merge 2 commits into
base: stable2409
Choose a base branch
from

Conversation

carlosala
Copy link
Contributor

At the moment, the delivery fees are being subtracted twice if the jit_withdraw mode is enabled, resulting in assets being trapped. This is an example of it.
This does not happen in 2412 anymore after #4834.

@carlosala carlosala requested a review from a team as a code owner January 16, 2025 11:20
@carlosala
Copy link
Contributor Author

/cmd prdoc --audience runtime_dev --bump patch

Copy link

Sorry, only members of the organization paritytech members can run commands.

@carlosala carlosala force-pushed the fix/reserve-jit-withdraw branch from 12daf63 to 8ed291c Compare January 16, 2025 11:32
@bkchr
Copy link
Member

bkchr commented Jan 16, 2025

CC @franciscoaguirre

@franciscoaguirre franciscoaguirre added the T6-XCM This PR/Issue is related to XCM. label Jan 16, 2025
@bkchr
Copy link
Member

bkchr commented Jan 16, 2025

@carlosala could we maybe get some test for this? :D (Not sure how the test infrastructure looks right now)

@carlosala
Copy link
Contributor Author

I'm not really sure either, what do you think @franciscoaguirre?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T6-XCM This PR/Issue is related to XCM.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants