-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Add RPC getStakeMinimumDelegation #26638
Conversation
4be5cb7
to
2dc19b3
Compare
pub async fn get_stake_minimum_delegation(&self) -> ClientResult<u64> { | ||
let instruction = solana_sdk::stake::instruction::get_minimum_delegation(); | ||
let transaction = Transaction::new_with_payer(&[instruction], None); | ||
let response = self.simulate_transaction(&transaction).await?; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Old way of simulate_transaction()
does not actually work due to checks in load_account()
.
Resolved! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r+ nit (alphabetizing RpcRequest variant)
Pull request has been modified.
* Add RPC getStakeMinimumDelegation * fixup! rpc * fixup rpc * fixup rpc client mock sender * fixup docs * pr: sort
Problem
There is not an RPC method to get the stake minimum delegation.
Summary of Changes
Add one 😊
Fixes #24690