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.

Sign up for GitHub

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

Jump to bottom

Metadata v15 Issues #2202

Closed
wilwade opened this issue Oct 28, 2024 · 0 comments · Fixed by #2204
Closed

Metadata v15 Issues #2202

wilwade opened this issue Oct 28, 2024 · 0 comments · Fixed by #2204
Assignees
@wilwade
Labels
bug Something isn't working

Comments

@wilwade
Copy link
Collaborator

wilwade commented Oct 28, 2024 •
edited
Loading

The upgrade to Metadata v15 is now causing this warning:

PORTABLEREGISTRY: Unable to determine runtime Call type, cannot inspect sp_runtime::generic::unchecked_extrinsic::UncheckedExtrinsic

However, there are currently no ill effects of it, nor additional errors. (even with DEBUG=*)

Possible Related Issues that might help:
@wilwade wilwade added the bug Something isn't working label Oct 28, 2024
@wilwade wilwade self-assigned this Oct 28, 2024
@wilwade wilwade mentioned this issue Oct 29, 2024
Merged
wilwade added a commit that referenced this issue Oct 30, 2024
@wilwade
Fix Missing runtime APIs for Metadata v15 (#2204)
9dffba0 
# Goal
The goal of this PR is to make sure that Metadata v15 has the correct
Runtime API call data

Closes #2202 

# Discussion

Why? I cannot be 100% sure. See the linked issues in #2202 for more
details.

From what I can tell, the way rust macros work with context means that
this macro is stuck being in the same file as the const that uses it is
`pub const VERSION`

# Specifics

- Updated Polkadot js API
- Fixed issue with the runtime APIs and the call in
`e2e/capacity/capacity_rpc.test.ts`. Issue noted in code comment
- Moved the runtime apis back into the runtime lib.rs
- Removed the manual definitions of runtimeApis 🎉 

## Testing

- `make start` && `subxt metadata --version 15 -f json | jq
.[1].V15.apis` (Before this result would have been empty, now you have
the runtime data!)
@bee344 bee344 mentioned this issue Dec 4, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wilwade wilwade

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Missing runtime APIs for Metadata v15 frequency-chain/frequency
1 participant
@wilwade