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

"Auto-compute" bug fixing #527

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Macha-orange
Copy link
Contributor

Did some fixing for the merge (incoherent message when options are changed) :

  • Clicking "Generate" button activate "auto-compute", so main feature isn't blocked if accidentally unchecked (and no more incoherent message).

  • All other buttons works only if "auto-compute" is checked. Including PhraseChanged()
    tabChanged() for BIP32/BIP44/BIP49/BIP84/BIP141
    delayedEntropyChanged()
    delayedSeedChanged()
    delayedRootKeyChanged()
    seedChanged()
    rootKeyChanged()
    calcForDerivationPath() for derivaton path change
    showMore() for adding more addresses

Did some fixing for the merge (incoherent message when options are changed) :

- Clicking "Generate" button activate "auto-compute", so main feature isn't blocked if accidentally unchecked (and no more incoherent message).

- All other buttons works only if "auto-compute" is checked. Including PhraseChanged()
tabChanged() for BIP32/BIP44/BIP49/BIP84/BIP141
delayedEntropyChanged()
delayedSeedChanged()
delayedRootKeyChanged()
seedChanged()
rootKeyChanged()
calcForDerivationPath() for derivaton path change
showMore() for adding more addresses
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant