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

Backport pallet info pub to SDK 1.14.0 #5202

Merged
merged 5 commits into from
Aug 15, 2024

Conversation

franciscoaguirre
Copy link
Contributor

Backporting #4976 to version 1.14.0.

The fields in PalletInfo were not pub in XCMv4 when they were for XCMv3.

JuaniRios and others added 2 commits July 31, 2024 16:37
v3 PalletInfo had the fields public, but not v4. Any reason why?
I need the PalletInfo fields public so I can read the values and do some
logic based on that at Polimec
@franciscoaguirre 

If this could be backported would be highly appreciated 🙏🏻

---------

Co-authored-by: Adrian Catangiu <[email protected]>
Co-authored-by: Oliver Tale-Yazdi <[email protected]>
@franciscoaguirre franciscoaguirre requested a review from a team as a code owner July 31, 2024 14:40
@franciscoaguirre franciscoaguirre added the T6-XCM This PR/Issue is related to XCM. label Jul 31, 2024
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: check-tracing
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6874942

prdoc/pr_4976.prdoc Outdated Show resolved Hide resolved
prdoc/pr_4976.prdoc Outdated Show resolved Hide resolved
@bkchr
Copy link
Member

bkchr commented Aug 12, 2024

@franciscoaguirre do we still need this?

@franciscoaguirre
Copy link
Contributor Author

@franciscoaguirre do we still need this?

Yes

@EgorPopelyaev EgorPopelyaev merged commit 3a7c3da into release-crates-io-v1.14.0 Aug 15, 2024
147 of 159 checks passed
@EgorPopelyaev EgorPopelyaev deleted the backport-pallet-info-pub branch August 15, 2024 14:55
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.

7 participants