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

mmr_generateProof params is not correct #5478

Closed
1 of 10 tasks
DaviRain-Su opened this issue Feb 8, 2023 · 3 comments · Fixed by #5479
Closed
1 of 10 tasks

mmr_generateProof params is not correct #5478

DaviRain-Su opened this issue Feb 8, 2023 · 3 comments · Fixed by #5479
Labels

Comments

@DaviRain-Su
Copy link

https://github.com/paritytech/substrate/blob/38c3e5ee10be760115a77a115ef0ac6ba381542b/client/merkle-mountain-range/rpc/src/lib.rs#L98

this rpc method params need to update.

  • I'm submitting a ...
  • Bug report
  • Feature request
  • Support request
  • Other
  • What is the current behavior and expected behavior?
  • What is the motivation for changing the behavior?
  • Please tell us about your environment:
  • Version:

  • Environment:

    • Node.js
    • Browser
    • Other (limited support for other environments)
  • Language:

    • JavaScript
    • TypeScript (include tsc --version)
    • Other
@jacogr
Copy link
Member

jacogr commented Feb 8, 2023

It is very outdated - there are a number of new RPC methods added as well.

PRs welcome to update https://github.com/polkadot-js/api/blob/master/packages/types/src/interfaces/mmr/rpc.ts

@polkadot-js-bot
Copy link

This issue has been open for 21 days with no activity and is not labelled as an enhancement. It will be closed in 7 days.

@polkadot-js-bot
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue if you think you have a related problem or query.

@polkadot-js polkadot-js locked as resolved and limited conversation to collaborators Apr 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants