Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Fix pallet-xcm Origin #3272

Merged
merged 1 commit into from
Jun 16, 2021
Merged

Fix pallet-xcm Origin #3272

merged 1 commit into from
Jun 16, 2021

Conversation

shawntabrizi
Copy link
Member

This places the custom Origin from pallet-xcm into the #[pallet] macro with the proper #[pallet::origin] tag.

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Jun 16, 2021
@shawntabrizi shawntabrizi added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels Jun 16, 2021
@shawntabrizi shawntabrizi merged commit d0ae2a5 into master Jun 16, 2021
@shawntabrizi shawntabrizi deleted the shawntabrizi-fix-xcm-pallet-origin branch June 16, 2021 20:57
@xlc
Copy link
Contributor

xlc commented Jun 17, 2021

Needs a change here as well? @jacogr
https://github.com/polkadot-js/api/blob/ae26aebdf80e79dfef486e18e3c83b3471127f80/packages/metadata/src/v13/toLatest.ts#L14-L19

s3krit pushed a commit that referenced this pull request Jun 17, 2021
@shawntabrizi @s3krit
fix xcm pallet origin (#3272)
60ee59d
ordian added a commit that referenced this pull request Jun 17, 2021
@ordian
Merge branch 'master' into ao-improved-gossip-topology
e59d267 
* master:
  Companion #9019 (max rpc payload override) (#3276)
  Implementers' Guide: Chain Selection (#3262)
  CLI: Add missing feature checking and check if someone passes a file (#3283)
  Export 'TakeRevenue' trait. (#3278)
  Add XCM Decode Limit (#3273)
  Allow Council to Use Scheduler (#3237)
  fix xcm pallet origin (#3272)
  extract determine_new_blocks into a separate utility (#3261)
  Approval checking unit tests (#3252)
  bridges: update finality-grandpa to 0.14.1 (#3266)
  malus - mockable overseer mvp (#3224)
  use safe math (#3249)
  Companion for #8920 (Control Staking) (#3260)
  Companion for #8949 (#3216)
@jacogr jacogr mentioned this pull request Jun 18, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@KiChjang KiChjang KiChjang approved these changes

Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shawntabrizi @xlc @KiChjang