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

Improve address management #4665

Closed
7 of 10 tasks
faboweb opened this issue Aug 6, 2020 · 14 comments · Fixed by #4669
Closed
7 of 10 tasks

Improve address management #4665

faboweb opened this issue Aug 6, 2020 · 14 comments · Fixed by #4669
Assignees

Comments

@faboweb
Copy link
Collaborator

faboweb commented Aug 6, 2020

Is your feature request related to a problem? Please describe.

Currently, it is not possible to have 2 accounts for one network
Currently, we have several separate lists of addresses (local, extension) this is confusing

Describe the solution you'd like

part 1

  • add extension accounts to address list
  • add local accounts to address list
  • reduce card sign in: remove sign in with account, remove sign in with extension
  • (optional) on extension accounts update, update also Lunie

part 2

  • add more context (stash, controller)
  • use names in the user menu
  • add max height and scroll bar to user menu address list

part 3

  • add button for forgetting an address
  • add security warning popup with pw confirmation if trying to forget a local account
  • add explanation popup for forgetting an extension account

Describe alternatives you've considered

Additional context

@Bitcoinera
Copy link
Contributor

remove sign in with extension

👀

@faboweb
Copy link
Collaborator Author

faboweb commented Aug 6, 2020

Remove the sign in with extension and local sign in options from the card sign in. They are not needed anymore when all addresses appear in the user menu.

@jbibla
Copy link
Collaborator

jbibla commented Aug 7, 2020

remove sign in with extension

he just means the option on the card sign in modal - because the list should auto-magically appear in the user menu

@Bitcoinera
Copy link
Contributor

ah ok, now I'm getting this task. Cool

@jbibla
Copy link
Collaborator

jbibla commented Aug 11, 2020

  • add more context (stash, controller)
  • use names in the user menu
  • add max height and scroll bar to user menu address list

@Bitcoinera
Copy link
Contributor

use names in the user menu

But this would be a PR in itself no? The ability to allow people to tag their addresses with names. We'd need a edit function. Should it be a modal opening for editing or how it should look like?

@faboweb
Copy link
Collaborator Author

faboweb commented Aug 13, 2020

We would use the names from the extension and the local accounts. For Ledger it would say "Ledger" for explore mode it would say "Explore"

@Bitcoinera
Copy link
Contributor

We would use the names from the extension and the local accounts. For Ledger it would say "Ledger" for explore mode it would say "Explore"

OK, I am happy to work on this. Next thing I am doing after applying reviews to #4607

@Bitcoinera
Copy link
Contributor

So how should I represent the account type (local, ledger, extension) ?

@Bitcoinera
Copy link
Contributor

Bitcoinera commented Aug 15, 2020

  • add more context (stash, controller)
  • use names in the user menu
  • add max height and scroll bar to user menu address list

Just accounted for all these things. But we should also merge luniehq/cosmos-keys#103 to be able to display the addressRole for all Polkadot accounts, not only the current one

@Bitcoinera
Copy link
Contributor

Bitcoinera commented Aug 15, 2020

I'd have this one left, which is quite cool: (optional) on extension accounts update, update also Lunie

EDIT: done

@Bitcoinera
Copy link
Contributor

To make the addresses computed for UserMenu reactive I just had to add a cache: false directive, as explained here: vuejs/vue#214 (comment) and here vuejs/vue#214 (comment)

@Bitcoinera
Copy link
Contributor

@jbibla I can help you with this one, adding the logic: "add button for forgetting an address"

@jbibla
Copy link
Collaborator

jbibla commented Aug 20, 2020

i opened a new issue to discuss "forgetting" #4751

This issue was closed.
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 a pull request may close this issue.

3 participants