-
Notifications
You must be signed in to change notification settings - Fork 54
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
Import wallet using seedphrase #137
Comments
Does that not work already for you? Wallets created via the CLI should show up in the UI. As the UI is just a frontend for the RPC API there shouldn't be any difference between wallets created via the CLI scripts and the UI. |
After importing wallet file across it shows up in the GUI which you can unlock it, but the UTXO's are not showing up. It's a different wallet file in bitcoin core, if i change it to the wallet that I've been using for joinmarket previously in the joinmarket.cfg the balance shows in cli. Tried using cli command of python wallet-tool.py -g 50 wallet.jmdat, and after that a bitcoin-cli rescanblockchain and then check balance and it still shows as 0 balance. update - using the rpcwallet file as part of the rescan command, waiting for the scan to finish /bitcoin-cli -rpcwallet=jam_default rescanblockchain that worked transactions and wallet balance can be seen. |
Just went through the same process as jase100k. Would be great to have a "rescan" option in wallet settings. Interested if this is possible since rescanblockchain isn't a JM RPC method. |
Link here for reference: JoinMarket-Org/joinmarket-clientserver#1082 |
This is not blocked anymore since JoinMarket-Org/joinmarket-clientserver#1461 is merged 💪 |
@editwentyone Would you be able to add an "Import Wallet" button and some views to Figma how you think the import process should look like? That'd be amazing. |
@theborakompanioni sure, could you give me a rough overview of the needed steps? just with bulletpoints what is actually needed from your side |
This is what currently is done when creating a wallet:
So, the above steps (and their corresponding copies) can be reused and basically become:
Feel free to design as you see fit. 🧡 |
on it… update: hot and greasy live from bitcoin Miami… @theborakompanioni: https://www.figma.com/file/kfejZJFlwBywvLEnPEmJo1/JoinMarket-UI?type=design&node-id=5959-66524&t=3wpjCPr8CTTzNAwh-11 questions:
|
During the first steps of implementing this, some thoughts came up:
|
let me know when I can test a first draft to experience it and also think about it :) |
Looks awesome. 🚀
Yes, I definitely need some more input as well and I am a little bit unsure of what the right approach is (e.g. currently I think, we have to "pre-generate" a bunch of addresses and then start the rescanning - which means, a user might has to rescan a couple of times.. which feels like bad UX). I'll let you know once I have the basic building blocks ready - currently it is all very rough, but I should be able to allocate some time soon. Things like "check of filename already taken during first screen" can be a follow-up feature request. I would not want to make reviewing the PR more complex than it already is (the wallet will just fail to be imported if the filename is already taken - same as within the current "create wallet" flow). |
Then we should get a hint until then, to name it accordingly. Also i thought about moving the date picker even a slide earlier, on the „active side“, where you actively insert your seed and choose a date. After that comes the „passive side“ to review only. Still sick, so Update will come later on figma unfortunately |
Existing power users should be able to move their wallets to this UI
The text was updated successfully, but these errors were encountered: