You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are some functionalities related to auths that we were considering adding to different commands, but IMO it could get a bit clunky to find those subcommands or flags, instead adding a new command sounds like a more clear user story.
Currently, I can see 2 primarily functionalities that we wanted to add, related to auths:
Signing auths (originally part of feat: add tx sign/send and Stellar trait #1406)
While it makes sense to add a new flag (--auth-only) to sign, moving it to a new command makes more sense it we want tx sign to have the same flags as other transaction signing commands. Moreover, not all transactions may have auth entries inside (in fact, most will not).
As such, we can add tx auth <xdr> --sign-with-* command that only sings auths.
This issue is stale because it has been assigned for 30 days with no activity. It will be closed in 30 days unless the stale label is removed, and the assignee is removed or updated.
There are some functionalities related to auths that we were considering adding to different commands, but IMO it could get a bit clunky to find those subcommands or flags, instead adding a new command sounds like a more clear user story.
Currently, I can see 2 primarily functionalities that we wanted to add, related to auths:
While it makes sense to add a new flag (
--auth-only
) to sign, moving it to a new command makes more sense it we wanttx sign
to have the same flags as other transaction signing commands. Moreover, not all transactions may have auth entries inside (in fact, most will not).As such, we can add
tx auth <xdr> --sign-with-*
command that only sings auths.--dry-run
for signing auths (Add a --dry-run option totx sign
#1486) nicely follows in line with ^auth show <xdr>
would be a great subcommand that lists all auths in this transaction, and shows which one is signed, which are not signedcc @janewang @willemneal @leighmcculloch
The text was updated successfully, but these errors were encountered: