-
Notifications
You must be signed in to change notification settings - Fork 1
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
XcmExecutorUtils pallet #5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the pallet looks very good. I would still wait for the upstream PR to get reviewed, just in case we are missing something by enabling the std... If we dont get it accepted we might just have to remove the genesis-build :(
We can always manually implement Serialize/Deserialize using a wrapper type, but I don't see why wouldn't they accept it |
polkadot-sdk PR has been approved and merged! paritytech/polkadot-sdk#3057 |
I think you can merge this PR unless @tmpolaczyk wants to add something else |
Go ahead |
* Assets filtering pallet on master * Changed structure to have a default and origins policies * Rename, tests, isTeleportFilter * Set teleport policies on genesis build * Test * Moved tests * Updated names * fmt * IKnit benchmarks * Removed Default for GenesisConfig. serde on std * Benchmarks and serde * fmt * Cleanup * Default policies mocks and tests * fmt
XcmExecutorUtils
is a utility pallet to help set the runtime parameters ofXcmExecutor
Currently it offers an intuitive, on-chain manager for trust policies on incoming assets though
IsReserveFilter
andIsTeleporterFilter
.