Skip to content
This repository has been archived by the owner on Jul 21, 2021. It is now read-only.

Enable WalletLink/WalletConnect #97

Merged
merged 9 commits into from
Jul 3, 2020
Merged

Enable WalletLink/WalletConnect #97

merged 9 commits into from
Jul 3, 2020

Conversation

JamesLefrere
Copy link
Contributor

@JamesLefrere JamesLefrere commented Jun 22, 2020

  • Clear the recently used wallet from local storage after disconnecting (so that
    users are not automatically reconnected on the next page load with the
    previous wallet type)
  • Upgrade use-wallet
  • Remove use-wallet types (now in the package itself)
  • Add REACT_APP_RPC_URL env var for connectors that need it
  • Add release link to footer
  • Add Github star button to footer

TODO

@JamesLefrere JamesLefrere changed the title WIP: Enable Walletlink/WalletConnect WIP: Enable WalletLink/WalletConnect Jun 22, 2020
@JamesLefrere JamesLefrere self-assigned this Jun 25, 2020
@JamesLefrere JamesLefrere changed the title WIP: Enable WalletLink/WalletConnect Enable WalletLink/WalletConnect Jun 25, 2020
@superduck35
Copy link
Contributor

Looks like use-wallet: 0.6.0 is out (https://github.com/aragon/use-wallet/releases/tag/v0.6.0)

👍

* Enable WalletConnect and WalletLink on mainnet
* Add missing logos for wallet types
* Reduce wallet icon size
* Clear the recently used wallet from local storage after disconnecting (so that users are not automatically reconnected on the next page load with the previous wallet type)
* Add migrations for local storage
* Remove `use-wallet` types (now in package)
* Add `REACT_APP_RPC_URL` env var for connectors that need it
* Upgrade `use-wallet`
* Add release link to footer
* Add Github star button to footer
@superduck35
Copy link
Contributor

Bug: 'Wrong network' behaviour has deprecated and does not provide accurate information

@superduck35 superduck35 merged commit 716e0a6 into master Jul 3, 2020
@superduck35 superduck35 deleted the enable-wallets branch July 3, 2020 11:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants