Skip to content

Comments

Updated error message for empty input#2665

Merged
ggreif merged 1 commit intocaffeinelabs:master-2665from
ehsan6sha:master
Jul 20, 2021
Merged

Updated error message for empty input#2665
ggreif merged 1 commit intocaffeinelabs:master-2665from
ehsan6sha:master

Conversation

@ehsan6sha
Copy link

Updated error message of empty input to be more explanatory per discussion below:

https://forum.dfinity.org/t/canister-xxx-trapped-explicitly-idl-error-empty-input/6002

Updated error message of empty input to be more explanatory per discussion below:
https://forum.dfinity.org/t/canister-xxx-trapped-explicitly-idl-error-empty-input/6002
@dfinity-ci
Copy link

This PR does not affect the produced WebAssembly code.

@ehsan6sha
Copy link
Author

Merging this can resolve the below issue:
#2666

@ehsan6sha
Copy link
Author

This PR does not affect the produced WebAssembly code.

This was just to make the output console error message more explanatory per the discussion I had with Joachim on the forum.

Copy link
Contributor

@nomeata nomeata left a comment

Choose a reason for hiding this comment

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

Thanks!

@ggreif ggreif added the automerge-squash When ready, merge (using squash) label Jul 20, 2021
@ggreif ggreif changed the base branch from master to master-2665 July 20, 2021 16:50
@ggreif ggreif merged commit 63a97f4 into caffeinelabs:master-2665 Jul 20, 2021
mergify bot pushed a commit that referenced this pull request Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge-squash When ready, merge (using squash)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants