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

Fix handling of the optional BIP39 passphrase flag #2489

Merged
merged 5 commits into from
Feb 5, 2024

Conversation

karbyshev
Copy link
Contributor

@karbyshev karbyshev commented Jan 31, 2024

  • Fix: respect the optional bip39-flag for key derivation.
  • Improvement: print the alias if the derived key already exists.

Based on v0.30.2

  • I have added a changelog
  • Git history is in acceptable state

@karbyshev karbyshev self-assigned this Jan 31, 2024
tzemanovic
tzemanovic previously approved these changes Jan 31, 2024
@Fraccaman Fraccaman mentioned this pull request Feb 1, 2024
@brentstone brentstone force-pushed the aleks/bip39-optional-derive branch from 0dccd02 to 07613e0 Compare February 1, 2024 19:06
brentstone added a commit that referenced this pull request Feb 1, 2024
* aleks/bip39-optional-derive:
  Ignore clippy to-many-arguments
  Add changelog
  Fix punctuation
  Print alias for the existing key
  Fix bip39-passphrase flag handling in wallet
@tzemanovic tzemanovic merged commit 89d080f into main Feb 5, 2024
14 of 15 checks passed
@tzemanovic tzemanovic deleted the aleks/bip39-optional-derive branch February 5, 2024 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants