pianity / arsnap

Arweave Snap for Metamask
20 stars 3 forks source link

Improve state updating strategy #15

Open noomly opened 2 years ago

noomly commented 2 years ago

The current strategy of fetching new data from the snap is very naive and feels buggy as some data doesn't update when we expect them to unless we refresh the page.

Steps to reproduce an example of this behavior:

  1. Uninstall Arsnap from Metamask if previously already configured
  2. Visit arsnap.org and click "Connect to Metamask"
  3. Without refreshing the page, navigate to the "Permissions" page
  4. Nothing is showing
  5. Refresh the page
  6. Permissions required by the wallet are now showing