-
Notifications
You must be signed in to change notification settings - Fork 75
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
Design for creating and combining multiple operations via CLI #1566
Comments
I think we should change In the future we may want sub-commands for adding other things, like This seems like a nice idea to me, but I'm not sure what the priority is as it seems like it would be usable only by some very advanced use cases, and folks can always use Stellar SDKs for advanced use cases. |
I think it's definitely useful for creating accounts since the generate can't fund on mainnet. And for payments (which yes can also be done via the SAC). Also I think set options could be useful. In general I think it is good for education and just being in alignment with the other tooling. The priority was that we made general transaction creation an OKR. |
Agreed, but that goal can be served without supporting sponsorship specifically, and without supporting multi-operation transactions. Do we specifically need those? |
A multi-operation transaction is a neat feature that we could work on in the future. For this quarter, I think being able to add classic ops is already a great win. |
Currently we are adding the ability to create operations transactions here: #1437
This adds a new
tx new
subcommand. Then each operation is a subcommand of this: e.g.By default this will sign and submit the transaction, but does also accept
--build-only
to return the built transaction.So to be able to add multiple operations into one transaction we would add a
tx add
subcommand. This would take a transaction envelope as input via stdin and then has the same interface asnew
, but in this case would add the operation to the passed in transaction. e.g.The text was updated successfully, but these errors were encountered: