feat: WalletActionHandler class & Wallet class, not integrated yet

EmerisHQ / demeris

Emeris web app

https://app.emeris.com/

Apache License 2.0

12 stars 2 forks source link

feat: WalletActionHandler class & Wallet class, not integrated yet #1680

Closed fl-y closed 2 years ago

fl-y commented 2 years ago

Description

WIP

Fixes: #1598

Feature flags

?VITE_FEATURE_USE_EMERIS_EXTENSION=true

Testing

Try doing anything that interacts with keplr after choosing Keplr in the wallet connect screen

Additional Details (Optional)

Connecting with the Emeris Extension doesn't work atm - whether these should be fixed in the Demeris side or the Extension side is somewhat a grey area. But in light of making the extension easier to integrate to other websites I think it's probably preferrable to fix the issues on the extension side if this works fine with Keplr.

cloudflare-pages[bot] commented 2 years ago

Deploying with Cloudflare Pages

Latest commit:	`c5466a8`
Status:	✅ Deploy successful!
Preview URL:	https://840867c7.emeris-app.pages.dev

View logs

github-actions[bot] commented 2 years ago

Visit the preview URL for this PR (updated for commit c61ccc6):

https://emeris-app--pr1680-feat-support-emeris-q1muc3dn.web.app

_{(expires Tue, 17 May 2022 19:13:13 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

faboweb commented 2 years ago

https://github.com/EmerisHQ/emeris-extension/pull/69 has getKey now

fl-y commented 2 years ago

Currently implemented with the premise that Keplr was the last connected wallet. Need to make it remember which wallet was last connected and auto-connect accordingly

faboweb commented 2 years ago

Caption is focusing on Keplr

faboweb commented 2 years ago

Need to allow Keplr first because of auto connect. Only then EE login is passing

faboweb commented 2 years ago

Let's merge and improve in consecutive PRs after the caption issue?