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

feat(timeout-height): allow define optional timeout height parameter #1489

Merged
merged 6 commits into from
Nov 7, 2023
Merged

feat(timeout-height): allow define optional timeout height parameter #1489

merged 6 commits into from
Nov 7, 2023

Conversation

janfabian
Copy link
Contributor

Introduce timeout height option for signing transactions (amino and direct) to support new POB/block-sdk modules.

Skip's documentation for details: https://docs.skip.money/searcher-docs

webmaster128
webmaster128 reviewed Oct 24, 2023
View reviewed changes
Copy link
Member

@webmaster128 webmaster128 left a comment

Choose a reason for hiding this comment

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

Great stuff, love it. High quality contribution ❤️

Some smaller changes. Then this can go in 0.32.

packages/cosmwasm-stargate/src/signingcosmwasmclient.spec.ts Show resolved Hide resolved
packages/cosmwasm-stargate/src/signingcosmwasmclient.spec.ts Outdated Show resolved Hide resolved
packages/cosmwasm-stargate/src/signingcosmwasmclient.spec.ts Outdated Show resolved Hide resolved
packages/cosmwasm-stargate/src/signingcosmwasmclient.ts Outdated Show resolved Hide resolved
packages/amino/src/signdoc.ts Outdated Show resolved Hide resolved
@webmaster128 webmaster128 added this to the 0.32.0 milestone Oct 24, 2023
@webmaster128
Copy link
Member

I just merged #1484 which turns all Long into bigints. Could you rebase this work onto current main?

webmaster128
webmaster128 reviewed Oct 26, 2023
View reviewed changes
Copy link
Member

@webmaster128 webmaster128 left a comment

Choose a reason for hiding this comment

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

Could you also add the parameter to signAndBroadcastSync and signAndBroadcast? I think signAndBroadcast is the highest level where people might need this. No need to write extra tests for those but just propagate the parameter.

@janfabian
Copy link
Contributor Author

Sure, I'm gonna add that tomorrow morning

webmaster128
webmaster128 approved these changes Nov 7, 2023
View reviewed changes
Copy link
Member

@webmaster128 webmaster128 left a comment

Choose a reason for hiding this comment

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

Beautiful, thank you!

@webmaster128 webmaster128 merged commit 0a9ab03 into cosmos:main Nov 7, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@webmaster128 webmaster128 webmaster128 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.32.0
Development

Successfully merging this pull request may close these issues.
2 participants
@janfabian @webmaster128