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

WalletAPI.Transfer does not work for sending from genesis account #210

Closed
devhawk opened this issue Apr 1, 2020 · 3 comments · Fixed by #213
Closed

WalletAPI.Transfer does not work for sending from genesis account #210

devhawk opened this issue Apr 1, 2020 · 3 comments · Fixed by #213

Comments

@devhawk
Copy link
Contributor

devhawk commented Apr 1, 2020

Attempting to send 1000 neo from genesis account to a standard address using a single node Neo3 chain (master branch). The consensus node wallet has two accounts, the node-specific account (NU78azX2nqbuDPthu5UGMaCXTVwpAkpvye) and the multi-sig account (NfC9EW6UJEbfX4TKCnqGf72AKHXUfR71ma). I am calling GetKey() on the multi-sig WalletAccount and passing it to WalletApi.Transfer. Transfer throws an exception Insufficient GAS in address: NU78azX2nqbuDPthu5UGMaCXTVwpAkpvye. That is the node-specific account address, NOT the multi-sig account address.

Stepping thru the transfer function, the exception is throwing in the TransactionManager.Sign() function. Sign is checking the gas balance, finding the gas balance to be zero and throwing.

@chenquanyu
Copy link
Contributor

Hi @devhawk
The WalletApi.Transfer is not ready for transferring assets from a multi-sig account now. You may refer to this document to construct a transaction.

@devhawk
Copy link
Contributor Author

devhawk commented Apr 2, 2020

Thanks @chenquanyu, I'll give that a try.

@devhawk
Copy link
Contributor Author

devhawk commented Apr 15, 2020

@chenquanyu I tried to follow the guidance you provided (with a local patch for #223) but SendRawTransaction throws an "Invalid" exception.

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 a pull request may close this issue.

2 participants