Fix minor issues

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

Hide file tree

Showing 2 changed files with 3 additions and 3 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/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)