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

[WIP] Allow metadata to be filtered to contain only the pallet information asked for #12369

Closed
wants to merge 2 commits into from

Conversation

jsdw
Copy link
Contributor

@jsdw jsdw commented Sep 27, 2022

Closes #12370

βœ„ -----------------------------------------------------------------------------

Thank you for your Pull Request! πŸ™

Before you submit, please check that:

  • Description: You added a brief description of the PR, e.g.:
    • What does it do?
    • What important points should reviewers know?
    • Is there something left for follow-up PRs?
  • Labels: You labeled the PR appropriately if you have permissions to do so:
    • A* for PR status (one required)
    • B* for changelog (one required)
    • C* for release notes (exactly one required)
    • D* for various implications/requirements
    • Github project assignment
  • Related Issues: You mentioned a related issue if this PR is related to it, e.g. Fixes #228 or Related #1337.
  • 2 Reviewers: You asked at least two reviewers to review. If you aren't sure, start with GH suggestions.
  • Style Guide: Your PR adheres to the style guide
    • In particular, mind the maximal line length of 100 (120 in exceptional circumstances).
    • There is no commented code checked in unless necessary.
    • Any panickers in the runtime have a proof or were removed.
  • Runtime Version: You bumped the runtime version if there are breaking changes in the runtime.
  • Docs: You updated any rustdocs which may need to change.
  • Polkadot Companion: Has the PR altered the external API or interfaces used by Polkadot?
    • If so, do you have the corresponding Polkadot PR ready?
    • Optionally: Do you have a corresponding Cumulus PR?

Refer to the contributing guide for details.

After you've read this notice feel free to remove it.
Thank you!

βœ„ -----------------------------------------------------------------------------

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Sep 27, 2022
@jsdw jsdw added B3-apinoteworthy C1-low PR touches the given topic and has a low impact on builders. labels Sep 27, 2022
@jsdw
Copy link
Contributor Author

jsdw commented Oct 12, 2022

Closing this; there is little appetite for such a change in the node side offhand.

@jsdw jsdw closed this Oct 12, 2022
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. C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support filtering the metadata to only contain information for specific pallets
1 participant