Fix minor issues

safe-global · Nov 7, 2023 · a604c5a · a604c5a
1 parent ea1ee95
commit a604c5a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/safe_cli/operators/safe_operator.py b/safe_cli/operators/safe_operator.py
@@ -294,7 +294,7 @@ def load_ledger_cli_owners(
                 print_formatted_text(HTML(f"{option} - <b>{address}</b> "))
 
             option = choose_option_question(
-                "Select the owner address", len(ledger_accounts) - 1
+                "Select the owner address", len(ledger_accounts)
             )
             if option is None:
                 return None
@@ -305,7 +305,7 @@ def load_ledger_cli_owners(
         print_formatted_text(
             HTML(
                 f"Loaded account <b>{address}</b> "
-                f'with balance={Web3.from_wei(balance, "ether")} ether'
+                f'with balance={Web3.from_wei(balance, "ether")} ether.\n'
                 f"Ledger account cannot be defined as sender"
             )
         )

diff --git a/safe_cli/prompt_parser.py b/safe_cli/prompt_parser.py
@@ -165,7 +165,9 @@ def load_cli_owners(args):
 
     @safe_exception
     def load_ledger_cli_owners(args):
-        safe_operator.load_ledger_cli_owners(args.derivation_path, args.legacy_accounts)
+        safe_operator.load_ledger_cli_owners(
+            derivation_path=args.derivation_path, legacy_account=args.legacy_accounts
+        )
 
     @safe_exception
     def unload_cli_owners(args):

diff --git a/safe_cli/utils.py b/safe_cli/utils.py
@@ -51,7 +51,7 @@ def choose_option_question(
 ) -> Optional[int]:
     if "PYTEST_CURRENT_TEST" in os.environ:
         return default_option  # Ignore confirmations when running tests
-    choices = f" [0-{number_options}] default {default_option}:"
+    choices = f" [0-{number_options-1}] default {default_option}: "
     reply = str(get_input(question + choices)).lower().strip() or str(default_option)
     try:
         option = int(reply)