hirosystems / connect

A library for building excellent user experiences with Stacks.
https://connect.stacks.js.org
MIT License
77 stars 41 forks source link

Remove sponsored option from doStxTransfer #168

Open friedger opened 3 years ago

friedger commented 3 years ago

I can't create a sponsored stx transfer transaction using doStxTransfer from connect-react. I can do a sponsored contract call.

I am not sure whether it is a problem with connect-react or with the stacks-wallet-web.

The UI shows tx fees and I guess that the tx would be broadcasted.

markmhendrickson commented 3 years ago

@agraebe did you happen to update connect-react when adding sponsored transaction functionality in addition to connect?

agraebe commented 3 years ago

oh, I didn't cover connect-react

friedger commented 3 years ago

Apparently, stx transfer txs can't be sponsored according to spec.

Then the options for doStxTransfer should not contain sponsored

agraebe commented 3 years ago

This is also something we should call out in our docs @pgray-hiro.

From SIP005:

Note that if a transaction contains a token-transfer payload, it MUST have only a standard authorization field. It cannot be sponsored.