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

algod REST API: Add support for algod /v2/teal/disassemble #702

Merged
merged 5 commits into from
Jan 18, 2023

Conversation

michaeldiamant
Copy link
Contributor

Adds support for /v2/teal/disassemble and corresponding tests from algorand/algorand-sdk-testing#251.

Since js-algorand-sdk endpoint support is not auto-generated, the PR adds support following conventions from https://github.com/algorand/js-algorand-sdk/blob/develop/src/client/v2/algod/compile.ts.

@michaeldiamant michaeldiamant marked this pull request as ready for review December 2, 2022 14:23
@algochoi algochoi self-requested a review January 18, 2023 18:11
.test-env Outdated Show resolved Hide resolved
algochoi and others added 4 commits January 18, 2023 13:59
* ABI:  Refactor ABI encoding test to round-trip (#701)

* bump version and add to changelog

* update README.md for new version

* Packaging: Improve source map and browser usage for external bundlers (#707)

* bump version and add to changelog

* update README.md for new version

* v2: Make breaking changes from v1 to v2.0.0  (#717)

* bump version and add to changelog

* update README.md for new version

* remove enhancement section of recent changelog

* Enhancement: Add foreign array objects to ATC `addMethodCall` (#725)

* Add foreign array objects to ATC addmethodcall

* Copy array value so that inputs are not modified

Co-authored-by: Michael Diamant <[email protected]>
Co-authored-by: Lucky Baar <[email protected]>
Co-authored-by: Jason Paulos <[email protected]>
Co-authored-by: Jack Smith <[email protected]>
Co-authored-by: Barbara Poon <[email protected]>
Copy link
Contributor

@algochoi algochoi left a comment

Choose a reason for hiding this comment

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

Tests look good and up to date with the develop branch

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.

2 participants