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

Fix the incorrect message name in messages.proto #17

Merged
merged 1 commit into from
Jul 6, 2018
Merged

Fix the incorrect message name in messages.proto #17

merged 1 commit into from
Jul 6, 2018

Conversation

quesalltime
Copy link
Contributor

@quesalltime quesalltime commented Jun 21, 2018

While generate the messages.proto , the message name of the return value for smart contract's constant function is incorrect.
For example ,
if a smart contract has a getName function which return a string ,
it will generate an unexpected GetNameReq message instead of generating GetNameResp message in the messages.proto.

While generate the messages.proto , the message name of the return value for smart contract's constant function is incorrect.
For example , 
if a smart contract has a `getBalance` function which return a string ,
it will generate an unexpected `GetBalanceReq` message instead of generating `GetBalanceResp` message in the messages.proto.
@quesalltime quesalltime changed the title Fix the incorrect message name of messages.proto Fix the incorrect message name in messages.proto Jun 21, 2018
@alanchchen alanchchen merged commit 177d400 into getamis:master Jul 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants