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

parachain origin to frame v2#3405

Merged
1 commit merged intoparitytech:masterfrom
ferrell-code:origin-framev2
Jul 5, 2021
Merged

parachain origin to frame v2#3405
1 commit merged intoparitytech:masterfrom
ferrell-code:origin-framev2

Conversation

@ferrell-code
Copy link
Copy Markdown
Contributor

relates: #2882

Following the upgrade guidelines here: https://crates.parity.io/frame_support/attr.pallet.html#upgrade-guidelines.

⚠️ Breaking Change ⚠️

From https://crates.parity.io/frame_support/attr.pallet.html#checking-upgrade-guidelines

storages now use PalletInfo for module_prefix instead of the one given to decl_storage: Thus any use of this pallet in construct_runtime! should be careful to update name in order not to break storage or to upgrade storage (moreover for instantiable pallet). If pallet is published, make sure to warn about this breaking change.

The was no decl_storage so no migration needed

Copy link
Copy Markdown
Contributor

@KiChjang KiChjang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it compiles, it works.

@shawntabrizi
Copy link
Copy Markdown
Member

@ferrell-code when this PR is complete, can you please include your polkadot or kusama address so I can open a treasury tip?

Thanks!

@ferrell-code
Copy link
Copy Markdown
Contributor Author

ferrell-code commented Jul 5, 2021

15zKd25HoSDPHCn1HjuuHTioXLT7PA2pZQ8gYjFsHW4r96qS

Or

DSSYqgJqdE6HmFAXpiq3y6eZ5FYZ3ZxmX3joRBkh345mxEN

thanks :)

If you have the time this week I think #8733 got buried in the pile of PRs
Edit2: nvm I see that you are working on #9090 probably easier for that to be merged first

Edit: I'm not sure why CI is hanging I believe it compiles fully

@gui1117 gui1117 added 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. labels Jul 5, 2021
@gui1117
Copy link
Copy Markdown
Contributor

gui1117 commented Jul 5, 2021

bot merge

@ghost
Copy link
Copy Markdown

ghost commented Jul 5, 2021

Waiting for commit status.

@ghost ghost merged commit 11f3ff6 into paritytech:master Jul 5, 2021
@gui1117
Copy link
Copy Markdown
Contributor

gui1117 commented Jul 5, 2021

Edit: I'm not sure why CI is hanging I believe it compiles fully

FYI, CI check that some labels are set on the PR, nothing on your side

@ferrell-code ferrell-code deleted the origin-framev2 branch July 5, 2021 17:57
ordian added a commit that referenced this pull request Jul 6, 2021
* master: (33 commits)
  Update all weights, add run_all_benches.sh script (#3400)
  Enable over-bridge-messaging in Rococo/Wococo runtime (#3377)
  paras.rs to FRAME V2 (#3403)
  Add XCM Tracing (#3353)
  Use MaxEncodedLen trait from new parity-scale-codec v2.2 (#3412)
  bump a bunch of deps in parity-common (#3402)
  Warn on low connectivity. (#3408)
  origin to frame v2 (#3405)
  Enable logging in the puppet worker (#3411)
  make it easier to dbg stalls (#3351)
  XCM `canonicalize` + `prepend_with` fix (#3269)
  cleanup stream polls (#3397)
  Staking Miner (#3141)
  Companion for Substrate#8953 (#3140)
  Bump version, specs & substrate in prep for v0.9.8 (#3387)
  Fix busy loops. (#3392)
  Minor refactor (#3386)
  add simnet tests (#3381)
  BEEFY: adjust gossip (#3372)
  Companion for #9193 (#3376)
  ...
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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

Development

Successfully merging this pull request may close these issues.

4 participants