-
Notifications
You must be signed in to change notification settings - Fork 15
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
Implement skycoin bip44 wallets #176
Implement skycoin bip44 wallets #176
Conversation
…coin_bip44_wallets
…not created wallets
…t README.md file
refs fibercrypto#256 - Add project badges in README.md
…ercrypto/FiberCryptoWallet into stdevHan_t256_add_converalls
src/ui/CreateLoadWallet.qml
Outdated
@@ -52,7 +52,7 @@ Item { | |||
dataModified() | |||
} | |||
} | |||
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove whitespace
@@ -16,6 +16,7 @@ Dialog { | |||
property alias mode: createLoadWallet.mode | |||
property alias name: createLoadWallet.name | |||
property alias seed: createLoadWallet.seed | |||
//property alias type: comboBoxWalletType.model[comboBoxWalletType.currentIndex].value |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove ?
@@ -31,15 +32,15 @@ Dialog { | |||
if (mode === CreateLoadWallet.Load){ | |||
scanA = 10 | |||
} | |||
walletModel.addWallet(walletManager.createEncryptedWallet(seed, name,textFieldPassword.text , scanA)) | |||
walletModel.addWallet(walletManager.createEncryptedWallet(seed, name,textFieldPassword.text, comboBoxWalletType.model[comboBoxWalletType.currentIndex].value, scanA)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
populate combobox from backend
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
... by using values returned by core.WalletSet.SupportedWalleTypes()
src/ui/GeneralStackView.qml
Outdated
@@ -102,12 +102,12 @@ Item { | |||
|
|||
onWalletCreationRequested: { | |||
stackView.replace(componentGeneralSwipeView) | |||
walletManager.createUnencryptedWallet(pageCreateLoadWallet.seed, pageCreateLoadWallet.name,0) | |||
walletManager.createUnencryptedWallet(pageCreateLoadWallet.seed, pageCreateLoadWallet.name, "deterministic" ,0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use constant / variable name
src/ui/GeneralStackView.qml
Outdated
} | ||
|
||
onWalletLoadingRequested:{ | ||
stackView.replace(componentGeneralSwipeView) | ||
walletManager.createUnencryptedWallet(pageCreateLoadWallet.seed, pageCreateLoadWallet.name,10) | ||
walletManager.createUnencryptedWallet(pageCreateLoadWallet.seed, pageCreateLoadWallet.name, "deterministic", 10) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use constant / variable name
…eralls Fixes fibercrypto#256 Add converalls
…ment_skycoin_bip44_wallets' into stdevMauricio1802_t148_Implement_skycoin_bip44_wallets
…the type of default wallets
…Acepted event handler
…etAvailableWalletTypes methods
Fixes #148
Changes:
Does this change need to mentioned in CHANGELOG.md?
yes
Requires testing
yes
Comments about testing , should you have some