Skip to content

Reconnect native wallet correctly#36

Merged
0xdef1cafe merged 1 commit into
mainfrom
native-wallet-reconnect
Sep 1, 2021
Merged

Reconnect native wallet correctly#36
0xdef1cafe merged 1 commit into
mainfrom
native-wallet-reconnect

Conversation

@0xdef1cafe
Copy link
Copy Markdown
Contributor

Notice

Before submitting a pull request, please make sure you have answered the following:

  • Have you followed the guidelines in our Contributing guide?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Do all new and existing tests pass? Does the linter pass?

Pull Request Type

  • 🐛 Bug fix (Non-breaking Change: Fixes an issue)
  • 🛠️ Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
  • 💅 New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #35

Description

when reconnecting a wallet after refreshing a browser it would initialize the wallet but not actually add it to the state in the context - this adds it to the state so it's actually available in the hook

Testing

Please outline all testing steps

  1. import/create a wallet
  2. refresh the page
  3. enter your password
  4. wallet should be available from useWallet hook

Screenshots (if applicable)

@0xdef1cafe 0xdef1cafe requested a review from technojak September 1, 2021 21:14
@0xdef1cafe 0xdef1cafe force-pushed the native-wallet-reconnect branch from 1b71c85 to c2594a0 Compare September 1, 2021 21:21
@0xdef1cafe 0xdef1cafe merged commit 757ef4b into main Sep 1, 2021
@0xdef1cafe 0xdef1cafe deleted the native-wallet-reconnect branch September 1, 2021 21:21
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 this pull request may close these issues.

Fix wallet reconnect hook

3 participants