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

Enhancement: Add State Proof support #370

Merged
merged 23 commits into from Aug 25, 2022
Merged

Enhancement: Add State Proof support #370

merged 23 commits into from Aug 25, 2022

Conversation

shiqizng
Copy link
Contributor

Add support for new state proof endpoints and transaction deserialization.

@shiqizng shiqizng changed the title StateProofs: Add State Proof support Enhancement: Add State Proof support Aug 19, 2022
@@ -248,6 +248,170 @@ def test_keyreg_txn_nonpart(self):
encoding.msgpack_encode(encoding.future_msgpack_decode(keyregtxn)),
)

def test_stateproof_txn(self):
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this one of the msgpack tests that the python cucumber test doesn't support?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, I extracted the txn object from v2_indexer_lookupBlock_transaction.json from encoded it using msgpacktool.

algosdk/v2client/algod.py Outdated Show resolved Hide resolved
algosdk/v2client/algod.py Outdated Show resolved Hide resolved
algosdk/v2client/algod.py Outdated Show resolved Hide resolved
algosdk/v2client/algod.py Outdated Show resolved Hide resolved
@shiqizng shiqizng requested a review from id-ms August 22, 2022 17:35
@shiqizng shiqizng requested a review from id-ms August 25, 2022 14:22
@shiqizng shiqizng merged commit 08efd34 into develop Aug 25, 2022
@winder winder deleted the shiqi/stateproof branch August 29, 2022 13:31
ahangsu added a commit that referenced this pull request Sep 2, 2022
* add check to desc so we dont output null if undefined (#368)

* Bumped version to v1.16.1

* Enhancement: Trim the indexer images and use the sandbox instead of custom dockers (#367)

Co-authored-by: algochoi <86622919+algochoi@users.noreply.github.com>
Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com>

* Bug-fix: Pass verbosity through to testing harness (#373)

* Enhancement: Add State Proof support (#370)

* add stateproof support

* Enhancement: Deprecating use of langspec  (#371)

* Bumped version to v1.17.0

* Update README.md

Co-authored-by: Zeph Grunschlag <tzaffi@users.noreply.github.com>

* Mergeback: Release v1.17.0b1 (#378)

* Bumped version
* Update README.md

* bumped version to v1.17.0

Co-authored-by: Ben Guidarelli <ben.guidarelli@gmail.com>
Co-authored-by: Barbara Poon <barbara.poon@algorand.com>
Co-authored-by: Zeph Grunschlag <tzaffi@users.noreply.github.com>
Co-authored-by: algochoi <86622919+algochoi@users.noreply.github.com>
Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com>
Co-authored-by: shiqizng <80276844+shiqizng@users.noreply.github.com>
Co-authored-by: Jack Smith <jack.smith@algorand.com>
Co-authored-by: Jack <87339414+algojack@users.noreply.github.com>
Co-authored-by: Lucky Baar <lucky.baar@algorand.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants