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

Treasury Pallet Account as destination for XCM fees on System Parachains #191

Conversation

muharem
Copy link
Contributor

@muharem muharem commented Feb 19, 2024

Designates the Treasury Pallet Account as the destination for XCM fees on System Parachains.

This ensures that assets accumulated in the pallet's account can be spent by Treasury Origins from Relay Chains.

@muharem muharem requested a review from Szegoo February 21, 2024 04:35
@muharem
Copy link
Contributor Author

muharem commented Mar 6, 2024

/merge

@fellowship-merge-bot
Copy link
Contributor

There was a problem running the action.

❌😵❌

Please find more information in the logs.

@ggwpez
Copy link
Member

ggwpez commented Mar 6, 2024

/merge

@ggwpez
Copy link
Member

ggwpez commented Mar 6, 2024

@Bullrich what does this mean? I see this log:

Warning: GraphqlResponseError: Request failed due to following response errors:
 - Pull request Pull request is in unstable status
Error: GraphqlResponseError: Request failed due to following response errors:
 - Pull request Pull request is in unstable status
Commenting: ### There was a problem running the action.

❌😵❌

Please find more information in the [logs](https://github.com/polkadot-fellows/runtimes/actions/runs/8[17](https://github.com/polkadot-fellows/runtimes/actions/runs/8172261940/job/22342205326#step:6:18)226[19](https://github.com/polkadot-fellows/runtimes/actions/runs/8172261940/job/22342205326#step:6:20)40).
Error: GraphqlResponseError: Request failed due to following response errors:
 - Pull request Pull request is in unstable status

@fellowship-merge-bot
Copy link
Contributor

There was a problem running the action.

❌😵❌

Please find more information in the logs.

@ggwpez ggwpez merged commit 35048df into polkadot-fellows:main Mar 6, 2024
29 of 30 checks passed
@Bullrich
Copy link
Contributor

Bullrich commented Mar 6, 2024

@Bullrich what does this mean? I see this log:

From the docs, it seems that unstable is when the PR can be merged but not all of the commits passed. It's clearly caused by this:
image

I'll modify the action so it can handle unstable actions too

@Bullrich
Copy link
Contributor

Bullrich commented Mar 6, 2024

Added issue to track this problem:

bkchr pushed a commit that referenced this pull request Mar 18, 2024
Asset Conversion instance for Polkadot Asset Hub and XCM Swap Weight
Trader for both Asset Hubs.


TODO:
- [x] update treasury account when this PR merged -
#191
- [x] define account deposit for pool assets, liquidity withdrawal fee,
pool setup fee;

---------

Co-authored-by: joe petrowski <[email protected]>
Co-authored-by: Sergej Sakac <[email protected]>
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.

6 participants