-
Notifications
You must be signed in to change notification settings - Fork 23
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
the return value of admin address #99
Comments
In fact, the idea is not to change the admin contract, but to change the address verification, attempt to remove the prefix Because the return value on the chain is indeed like this, I feel that as a tool, I can't change the return data on the chain at will. |
well, maybe a -h parameter can make this requirement meaningful
漂流 <[email protected]>于2019年1月30日 周三17:48写道:
In fact, the idea is not to change the admin contract, but to change the
address verification, attempt to remove the prefix 00... of the address.
Because the return value on the chain is indeed like this, I feel that as
a tool, I can't change the return data on the chain at will.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#99 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAwjiB_JB3tbjS6sFBR1daXveMIfk1j0ks5vIWp-gaJpZM4aZv4I>
.
--
Regards,
Timmy Zhang
Sent from Gmail Mobile
|
Well, this really can be considered. However, this may require a lot of work. It may need to refactor some code. |
the image above should have described the issue clearly. well, i'm not quite familiar with rust grammar, i guess this line (cita-cli/src/cli/contract_command.rs) should be modified:
make the return value remove the prefix 00000..., maybe some method named wrap()?
The text was updated successfully, but these errors were encountered: