Skip to content
This repository has been archived by the owner on Nov 17, 2022. It is now read-only.

Integrate minimal wallet connect implementation #1

Open
TimDaub opened this issue Nov 19, 2021 · 0 comments
Open

Integrate minimal wallet connect implementation #1

TimDaub opened this issue Nov 19, 2021 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@TimDaub
Copy link
Member

TimDaub commented Nov 19, 2021

Scope

  • Currently, messages can only be signed with metamask as it's the only provider inserted into the page
  • However, e.g. on mobile e.g. WalletConnect would be required

Deliverable

  • Give user two buttons "Connect with MM", "Connet with Wallet Connect"
  • To implement WalletConnect, find the most minimal footprint integration. DON'T just dump an arbitrary NPM package in the dependencies. Try to make the wallet connect integration as small in size as possible
  • Before committing as PR, see consensus here in the issue

time estimate

  • estimate includes familiarizing oneself with wallet connect and the component's structure
  • checking for the most minimal approach towards implementation
  • doing the actual work
  • 4h
@TimDaub TimDaub added enhancement New feature or request good first issue Good for newcomers labels Nov 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant