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

Enable --sign-with-lab to return the signed tx to CLI for submission #1640

Open
janewang opened this issue Oct 1, 2024 · 3 comments
Open

Comments

@janewang
Copy link
Contributor

janewang commented Oct 1, 2024

Follow up work from this issue: #1428. Enable a signed tx to be returned to the CLI for submission

@github-project-automation github-project-automation bot moved this to Backlog (Not Ready) in DevX Oct 1, 2024
@leighmcculloch
Copy link
Member

leighmcculloch commented Oct 2, 2024

This is critical for the --sign-with-lab feature to be usable by the average contract developer.

Iirc the main reason we wanted to do sign-with-lab was so that folks could sign with their wallets. Most users will only sign as part of the regular flow of doing operations like stellar contract deploy, and in that flow we can only offer sign-with-lab if the tx is returned, because they are sometimes multi-step flows where multiple transactions are submitted.

Power users will be able to do stellar contract deploy --build-only | stellar tx simulate | stellar tx sign --sign-with-lab, but that flow is really intended for power users, and less-power users using wallets should be able to sign with their wallet without going the power user route by doing stellar contract deploy --sign-with-lab.

@willemneal
Copy link
Member

Also as is right no, users won't be able to stellar contract install because the current solution is opening a link, which cannot handle the size of the upload transaction. And currently I assume that many devs use --wasm with deploy which requires an install. (I wish I had pushed more to remove that so feature so deploy only did one thing).

I also think that less power users would want to make use of their browser wallets and would like signing and sending from lab.

Perhaps we should poll users on discord. Also @kalepail 's work on passkeys might offer a way to create the passkey with the browser for lab once and then users could use it via the CLI, but that's perhaps more advanced (and we might not even need the browser to create the passkey?)

Copy link
Contributor

github-actions bot commented Dec 4, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo (Ready for Dev)
Development

No branches or pull requests

4 participants