Skip to content

Enhancement: allowing zero-length static array#698

Merged
ahangsu merged 4 commits into
developfrom
zero-length
Nov 29, 2022
Merged

Enhancement: allowing zero-length static array#698
ahangsu merged 4 commits into
developfrom
zero-length

Conversation

@ahangsu
Copy link
Copy Markdown
Contributor

@ahangsu ahangsu commented Nov 28, 2022

Fixes #497

Comment thread .test-env Outdated
Copy link
Copy Markdown
Contributor

@michaeldiamant michaeldiamant left a comment

Choose a reason for hiding this comment

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

@ahangsu Looks correct - Thanks for bringing the fix into js-algorand-sdk!

Comment thread tests/10.ABI.ts
Copy link
Copy Markdown
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.

LGTM!

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.

ABI decoder does not allow zero-length arrays

3 participants