Skip to content

Fix: improve error message for mismatched args#511

Merged
winder merged 2 commits intoalgorand:developfrom
barnjamin:atc-methodarg-err-msg
Mar 24, 2023
Merged

Fix: improve error message for mismatched args#511
winder merged 2 commits intoalgorand:developfrom
barnjamin:atc-methodarg-err-msg

Conversation

@barnjamin
Copy link
Copy Markdown
Contributor

@barnjamin barnjamin commented Mar 1, 2023

Changing the error message to be more accurate

Comment thread src/main/java/com/algorand/algosdk/transaction/MethodCallParams.java Outdated
…s.java

Co-authored-by: Michael Chuang <github@michaeltchuang.com>
@barnjamin barnjamin marked this pull request as ready for review March 2, 2023 11:14
@barnjamin barnjamin changed the title improve error message for mismatched args Fix: improve error message for mismatched args Mar 2, 2023
@barnjamin barnjamin requested a review from winder March 23, 2023 18:20
Copy link
Copy Markdown
Contributor

@winder winder left a comment

Choose a reason for hiding this comment

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

LGTM

@winder winder merged commit 20e1ed5 into algorand:develop Mar 24, 2023
@barnjamin barnjamin deleted the atc-methodarg-err-msg branch March 24, 2023 14:48
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.

3 participants