Add connect wallet button #227

radchukd · 2021-09-06T13:43:25Z

I fixed wallet loading and added the "Connect a wallet" button along with some refactoring.

… "connect a wallet" button to drawer

apbendi

Hey @radchukd, looking good! Nice work.

One thing I did notice is clicking on my address after connecting, which used to allow the user to change wallets, now produces an error:

frontend/src/store/wallet.ts

frontend/src/layouts/BaseLayout.vue

mds1

LGTM, thanks @radchukd!

@apbendi I'll wait for your final review before merging

radchukd added 3 commits September 2, 2021 22:21

add connect wallet button, fix wallet loading

9df82fb

add rounded prop to BaseButton, remove onboard reinitialization, move…

15a612c

… "connect a wallet" button to drawer

add connectWallet function

2e2e779

apbendi requested a review from mds1 September 7, 2021 11:29

apbendi reviewed Sep 7, 2021

View reviewed changes

radchukd added 2 commits September 7, 2021 15:41

fix wallet changing, add wallet storing after logging in

0a1bf4f

fix network switching loading

e5c2a22

mds1 reviewed Sep 10, 2021

View reviewed changes

frontend/src/store/wallet.ts Outdated Show resolved Hide resolved

mds1 reviewed Sep 10, 2021

View reviewed changes

frontend/src/store/wallet.ts Outdated Show resolved Hide resolved

mds1 reviewed Sep 10, 2021

View reviewed changes

frontend/src/layouts/BaseLayout.vue Show resolved Hide resolved

radchukd added 2 commits September 10, 2021 21:11

fix connectWallet recalling

4c3b9ca

add hasSelected condition

52f577a

mds1 approved these changes Sep 12, 2021

View reviewed changes

apbendi merged commit 52f577a into ScopeLift:master Sep 17, 2021

Provide feedback