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

Support scripting #36

Closed
2 tasks done
Uxio0 opened this issue Dec 1, 2020 · 3 comments · Fixed by #419 or #426
Closed
2 tasks done

Support scripting #36

Uxio0 opened this issue Dec 1, 2020 · 3 comments · Fixed by #419 or #426
Assignees

Comments

@Uxio0
Copy link
Member

Uxio0 commented Dec 1, 2020

Currently the only way to use the safe-cli is using the interactive mode. For example, to send some ether a user must run the app, load the owners and then call send_ether and press intro.

An automatic way to do it, like python safe_cli --node <node_url> --private-key <owner_private_key_1> --private-key <owner_private_key_2> send-ether --wei <ether_in_wei> --to <address>

Needed for a first approach:

  • Being able to execute transactions or batch transactions
  • Being able to load and execute transactions from a file (the transaction builder in the UI can export files, use the same format)
@Uxio0 Uxio0 self-assigned this Dec 1, 2020
@ademidun
Copy link

ademidun commented Feb 3, 2023

Agreed. Especially, because the interactive mode requirement also makes it unusable in remote servers like Google Colab. I made a Jupyter notebook to make it easy for users to run the safe-cli from the browser since some users won't be able to install the command line themselves.

I get the following error:

WARNING: your terminal doesn't support cursor position requests (CPR).

Which I think is related to the lack of interactivity. So this causes the load safe command to hang and never complete.

I think this is a high priority fix as many issues that users face through UI can be solved through CLI and many users may not be able to use the CLI but they can use the notebook.

@Uxio0
Copy link
Member Author

Uxio0 commented Jun 10, 2024

Take a look at https://typer.tiangolo.com/

@JagoFigueroa
Copy link

Tested and verified, many thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants