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

Expose multiple merkle proofs #198

Closed
ethanfrey opened this issue Aug 1, 2017 · 2 comments
Closed

Expose multiple merkle proofs #198

ethanfrey opened this issue Aug 1, 2017 · 2 comments

Comments

@ethanfrey
Copy link
Contributor

References tendermint/merkleeyes#30

  • These have to be available through the abci_query interface.
  • Support non-existence for keys type
  • Add other paths- "/list", "/first", "/last" with proofs for other query types
@ethanfrey ethanfrey added the 0.7 label Aug 2, 2017
@ethanfrey
Copy link
Contributor Author

Maybe we need to expand the abci interface to do so properly for range queries: tendermint/abci#94

@jaekwon jaekwon removed the 0.7 label Jan 14, 2018
@adrianbrink adrianbrink mentioned this issue Feb 26, 2018
Closed
@adrianbrink
Copy link
Contributor

closed in favour of #341

alexanderbez added a commit that referenced this issue May 26, 2022
@mattverse @ValarDragon @alexanderbez
feat: add clawback vesting account (#178)
8692bad 
* Add proto for Clawback Vesting Account

* Change field to int

* Change field to int64

* add place holder for clawback

* add handler and basic msgs methods

* add clawback vesting account logic

* improvements

* fix: Dev/clawback test debug (#198)

* add tests

* add clawback before starttime test

* clean imports

* Printlns for the clawback testing

* Fix tests and logic

* Fix the gas exhaustion vuln

* Remove printlns

* Delete extraneous account set

* chore: add table driven test cases to clawback (#199)

* add table driven tests

* Modify test cases

* More detail in error message

* Update x/auth/vesting/types/vesting_account_test.go

* Update x/auth/vesting/types/vesting_account_test.go

Co-authored-by: Dev Ojha <[email protected]>

Co-authored-by: Dev Ojha <[email protected]>
Co-authored-by: Dev Ojha <[email protected]>

* fix: cherry pick agoric's cosmos sdk#207 (#203)

* cherry pick compute endtime fix

* Merge branch 'osmosis-main' into mattverse/clawback

Co-authored-by: mattverse <[email protected]>
Co-authored-by: Matt, Park <[email protected]>

* feat: add clawback cli and tests (#207)

* add clawback cli and tests

* Update x/auth/vesting/client/cli/tx.go

Co-authored-by: Aleksandr Bezobchuk <[email protected]>

* Update x/auth/vesting/client/cli/tx.go

Co-authored-by: Aleksandr Bezobchuk <[email protected]>

* Update x/auth/vesting/client/cli/tx.go

Co-authored-by: Aleksandr Bezobchuk <[email protected]>

* minor fix

* Minor fix

* Update x/auth/vesting/client/cli/cli_test.go

Co-authored-by: Aleksandr Bezobchuk <[email protected]>

Co-authored-by: Aleksandr Bezobchuk <[email protected]>

* Change from code review from Alex and Dev

Co-authored-by: Dev Ojha <[email protected]>
Co-authored-by: Aleksandr Bezobchuk <[email protected]>
mmsqe added a commit to mmsqe/cosmos-sdk that referenced this issue Feb 2, 2024
@mmsqe
Merge pull request cosmos#198 from mmsqe/release/v0.47.x_node_uri
c887e86 
Problem: NodeURI is not set for clientCtx in test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jaekwon @ethanfrey @adrianbrink