Skip to content

Conversation

@DaughterOfMars
Copy link
Contributor

Description

Adds two high level functions for staking to the transaction builder.

Closes #311

@DaughterOfMars DaughterOfMars linked an issue Oct 27, 2025 that may be closed by this pull request
Thoralf-M
Thoralf-M previously approved these changes Oct 27, 2025
@thibault-martinez
Copy link
Member

CI failing

@thibault-martinez
Copy link
Member

I am fine with this version but the issue suggested

Maybe one should just provide an amount instead of objects as parameter and then the amount is split from the gas coin and staked?

Have we considered it?

@DaughterOfMars
Copy link
Contributor Author

I am fine with this version but the issue suggested

Maybe one should just provide an amount instead of objects as parameter and then the amount is split from the gas coin and staked?

Have we considered it?

Didn't see that, but it's a good idea. I'll change it.

@thibault-martinez thibault-martinez dismissed Thoralf-M’s stale review October 27, 2025 16:31

Changed enough since review

@thibault-martinez thibault-martinez merged commit b7fe7ba into sdk-bindings Oct 28, 2025
14 checks passed
@thibault-martinez thibault-martinez deleted the feat/high-level-stake branch October 28, 2025 08:25
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.

Tx builder: add high level un/staking methods

4 participants