Skip to content
This repository has been archived by the owner on Sep 5, 2020. It is now read-only.

error/infinite import of wallet-file #714

Closed
luclu opened this issue May 25, 2016 · 5 comments · Fixed by #1598
Closed

error/infinite import of wallet-file #714

luclu opened this issue May 25, 2016 · 5 comments · Fixed by #1598
Milestone

Comments

@luclu
Copy link
Contributor

luclu commented May 25, 2016

The import of non-presale-wallets will cause a stalling/infinite import dialog.
Affected wallets (web3-secret-storage version 1 to 3):

We could also change the menu-label from import Pre-sale Accounts to import Wallet.

@alexvandesande
Copy link
Collaborator

@luclu what's the status on this?

@luclu
Copy link
Contributor Author

luclu commented Aug 29, 2016

There is an outstanding design decision - I would like to get your input at.

As you know the import-wallet view will ask for a password after the drop event, but as
web3-secret-storage-files just need to be copied it is not needed. Which gives me three possibilites:

  1. let the user input the password regardless of the wallet type and use the code at ipc_import-etc to decide on the procedure
  2. get some js to figure out the wallet type before proceeding to the password screen
  3. use user input elements to let the user decide on the wallet type
  4. create a second menuitem for web3-files

I would clearly prefer 2. but can't figure out a way to include fs to read and evaluate the json file for its type within meteor's onboarding js code (clientside).
Which leaves me with new ipc-command and evaluation before the user might be prompt for a password.

@hiddentao
Copy link
Contributor

(2) is the most elegant solution, but (3) is a decent fallback.

@evertonfraga evertonfraga modified the milestones: 0.8.3, 0.8.4 Sep 22, 2016
@luclu
Copy link
Contributor Author

luclu commented Sep 24, 2016

Finally implemented basic functionality of (2) on the airplane today. ;)
Import of web3 as well as presale files is now possible, while the former will directly show the importing animation as a dummy for 750ms.

@evertonfraga evertonfraga modified the milestones: 0.8.4, 0.8.5 Sep 27, 2016
@evertonfraga evertonfraga removed this from the 0.8.5 milestone Oct 7, 2016
@evertonfraga evertonfraga modified the milestones: 0.9, 0.8.5 Oct 7, 2016
@luclu luclu mentioned this issue Jan 17, 2017
5 tasks
@luclu luclu removed their assignment Mar 19, 2017
@evertonfraga evertonfraga modified the milestones: 0.9, 0.9.0 Mar 22, 2017
@lock
Copy link

lock bot commented Mar 31, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked and limited conversation to collaborators Mar 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants