search

NewEconoLab / TeemoWallet

Chrome Extension for NEO
4 stars 0 forks source link

Teemo.NEO.send() returning error #1

Closed vincentgeneste closed 5 years ago

vincentgeneste commented 5 years ago

Hello!

Trying to implement Teemo dAPI and I'm having issue with the send() function. I tried with your own example, I do have enough GAS in my wallet, but for some reason it's not working.

I get the popup once I do a send(), but there's an error in the console and the popup looks empty.

Screen Shot 2019-03-19 at 20 31 40 Screen Shot 2019-03-19 at 20 29 30
QingmingZi commented 5 years ago

@vincentgeneste Thank you very much for your feedback. This method is one of the few that we haven't fully completed yet. Expected to be completed tomorrow

QingmingZi commented 5 years ago

@vincentgeneste send() is OK, Now

vincentgeneste commented 5 years ago

I do get the popup now, but I still get an error when trying to send something, and it's not very descriptive why it failed?

Screen Shot 2019-03-20 at 10 42 52
QingmingZi commented 5 years ago

@vincentgeneste Recommended to get the latest code test (testnet) image

vincentgeneste commented 5 years ago

Is it only working on testnet for now? Will give it a try tonight

vincentgeneste commented 5 years ago

Ok just tested and it works when using

"asset": "602c79718b16e442de58778e148d0b1084e3b2dffd5de6b7b16cee7969282de7"

But when I use "asset":"GAS" like in your documentation, it does not.

QingmingZi commented 5 years ago

Sorry, this is a problem with the documentation. The asset ID should always be used instead of the asset name or asset unit. Because we believe that the asset name or asset unit is not unique and is not suitable as an argument to the asset field. https://dapi.nel.group/en/#send has been fixed

vincentgeneste commented 5 years ago

Ok got it if you updated the documentation that’s fine :-)