paritytech / parity-extension

Parity Chrome Extension
GNU General Public License v3.0
37 stars 14 forks source link

Node status in Popup // Option page #49

Closed ngotchac closed 7 years ago

ngotchac commented 7 years ago

Closes #9

Better Account Card in the Pop-Up + Ground for node connection status

Created an Option page where one can:

  1. Enable/Disable the Identity Extraction/Augmentation
  2. Enable/Disable the Web3 Integration
  3. Modify the node and dapps URL
  4. Modify the Lookup-Service URL (as a fallback)

~Need #48 to be merged before being able to finish this one.~

Some options are hidden by default, but available via an Advanced Options checkbox

ngotchac commented 7 years ago

@tomusdrw Addressed PR Grumbles :)

tomusdrw commented 7 years ago

Seems that it also closes #52

tomusdrw commented 7 years ago

Just did some tests:

  1. When you open a page while not being connected to the node it displays identities correctly, but when you run the node it stops displaying identities: screenshot from 2017-02-10 08-33-42

  2. Red icon is not drawn right after installation: screenshot from 2017-02-10 08-42-07

ngotchac commented 7 years ago

@tomusdrw Was this the same chain ? By default, without node, it fetches the identities from the main net. If you run a node on the test net, there will be some cache issues I think :/

tomusdrw commented 7 years ago

I'm not sure now, it's possible it was different chain. Can't we re-compute identities?

ngotchac commented 7 years ago

Yes, this cache needs to be clear automatically on network/lookup URL change

ngotchac commented 7 years ago

@tomusdrw Okay, the cache should work better now (caching per retrieving method). The little connection icon should work better and the network name is displayed