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

Expose parachain-staking methods to RPC in all runtimes #350

Merged
merged 9 commits into from
Nov 23, 2023

Conversation

gianfra-t
Copy link
Contributor

@pendulum-chain/product: This PR adds changes to the node client code that require a redeployment of the collator nodes to take effect.
Please ensure that the collator nodes are redeployed after this PR is merged.

This PR adds get_unclaimed_staking_rewards and get_staking_rates methods from parachain-staking to the RPC configuration and allows them to be called.

@gianfra-t gianfra-t changed the title exposed get_unclaimed_staking_rewards and get_staking_rates as rpc calls Expose parachain-staking methods to RPC in all runtimes Nov 17, 2023
@gianfra-t gianfra-t marked this pull request as ready for review November 20, 2023 12:41
@gianfra-t gianfra-t requested a review from a team November 20, 2023 12:41
pallets/parachain-staking/rpc/Cargo.toml Outdated Show resolved Hide resolved
pallets/parachain-staking/rpc/runtime-api/Cargo.toml Outdated Show resolved Hide resolved
res/amplitude-spec-raw.json Outdated Show resolved Hide resolved
runtime/amplitude/src/lib.rs Show resolved Hide resolved
runtime/amplitude/Cargo.toml Outdated Show resolved Hide resolved
runtime/pendulum/src/lib.rs Outdated Show resolved Hide resolved
pallets/parachain-staking/rpc/runtime-api/src/lib.rs Outdated Show resolved Hide resolved
@prayagd
Copy link
Collaborator

prayagd commented Nov 21, 2023

@ebma should we already create ticket for this on the portal?

@ebma
Copy link
Member

ebma commented Nov 21, 2023

@prayagd a ticket to employ the exposed functions? Sure, we can create that if it does not yet exist. Note that we need to redeploy the collator nodes on each network for this to take effect though, so merging this PR does not immediately unblock the work on the Portal.

Copy link
Member

@ebma ebma left a comment

Choose a reason for hiding this comment

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

Ready for merge once the conflicts are resolved

@ebma
Copy link
Member

ebma commented Nov 22, 2023

CI is failing, let's fix that first

@gianfra-t
Copy link
Contributor Author

Now it should be good!

@gianfra-t gianfra-t force-pushed the add-staking-rpc-calls branch from b6614ae to 548e843 Compare November 22, 2023 16:09
@gianfra-t gianfra-t requested a review from ebma November 22, 2023 21:00
@gianfra-t gianfra-t merged commit 46531bf into main Nov 23, 2023
2 checks passed
@gianfra-t gianfra-t deleted the add-staking-rpc-calls branch November 23, 2023 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants