spesmilo / spesmilo/electrum

Inconsistent Default Behavior between electrum getbalance and electrum load_wallet command

Open
#6,360 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
8.6k
Forks
3.5k
Avg merge
2d 8h
Merged PRs (30d)
61

Description

If in a folder other than the default directory
attempting the following sequence will fail on line 3 and work on ALL other lines

electrum create --wallet wallet_name --password wallet_password
electrum load_wallet --wallet wallet_name --password wallet_password 
electrum getbalance --wallet wallet_name
electrum close_wallet --wallet wallet_name

from a consistency standpoint, anywhere that create_wallet by default creates wallets, SHOULD also be a directory that getbalance attempts to scan to search for wallets to load. This contract is upheld by the load_wallet and close_wallet functions, so should be held across the board.

Image linked: https://imgur.com/a/2gXRtgP

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the four-command sequence from the issue, especially when the wallet is outside the default directory. Start by tracing the CLI entry points for create, load_wallet, getbalance, and close_wallet. Done means getbalance searches the same wallet location that create and the other wallet commands use by default.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.