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

Add send coin ability as command line parameters #1738

Closed
wants to merge 1 commit into from

Conversation

sweep-ai[bot]
Copy link
Contributor

@sweep-ai sweep-ai bot commented Aug 7, 2023

Description

This PR adds the ability to send coins as command line parameters in the naruno/cli/main.py file. It introduces three new command line arguments: --password, --recipient, and --amount. These arguments can be used to provide the necessary information for sending coins without starting the menu.

Summary

  • Added necessary imports for argparse and getpass in naruno/cli/main.py.
  • Modified the arguments() function to define the --password, --recipient, and --amount arguments using argparse.ArgumentParser.add_argument().
  • Modified the menu() function to check if the --sendcoin argument is used and retrieve the values of the password, recipient, and amount arguments.
  • Updated the show_menu() function to reflect the new command line options and provide information about how to use them.

Fixes #1737.


To checkout this PR branch, run the following command in your terminal:

git checkout sweep/add-send-coin-command-line-params

To get Sweep to edit this pull request, leave a comment below or in the code. Leaving a comment in the code will only modify the file but commenting below can change the entire PR.

@sweep-ai sweep-ai bot added the sweep Assigns Sweep to an issue or pull request. label Aug 7, 2023
@sweep-ai sweep-ai bot temporarily deployed to Tests August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Codecov August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to LINT Tests August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Codecov August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Codecov August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Tests August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@sweep-ai sweep-ai bot temporarily deployed to Builds August 7, 2023 16:48 Inactive
@viezly
Copy link

viezly bot commented Aug 7, 2023

Changes preview:

Legend:

👀 Review pull request on Viezly

@onuratakan onuratakan closed this Aug 7, 2023
@sweep-ai sweep-ai bot temporarily deployed to UNIT Tests August 7, 2023 16:55 Inactive
@sweep-ai sweep-ai bot temporarily deployed to FUNCTIONAL Tests August 7, 2023 16:55 Inactive
@sweep-ai sweep-ai bot temporarily deployed to FUNCTIONAL Tests August 7, 2023 16:55 Inactive
@sweep-ai sweep-ai bot temporarily deployed to UNIT Tests August 7, 2023 16:55 Inactive
@sweep-ai sweep-ai bot temporarily deployed to UNIT Tests August 7, 2023 16:59 Inactive
@sweep-ai sweep-ai bot temporarily deployed to UNIT Tests August 7, 2023 16:59 Inactive
@sweep-ai sweep-ai bot temporarily deployed to FUNCTIONAL Tests August 7, 2023 16:59 Inactive
@sweep-ai sweep-ai bot temporarily deployed to FUNCTIONAL Tests August 7, 2023 16:59 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sweep: Adding send coin ability to parameters of cli
1 participant