search

Giveth / giveth-docs

Documentation and Guides for Giveth.io
GNU General Public License v3.0
37 stars 33 forks source link

Add new wallets to Documentation #204

Open franculio opened 8 months ago

franculio commented 8 months ago

Going through the documentation I noticed that we could update the new wallets added in the last version of the Dapp released.

Private Zenhub Image

@divine-comedian could you confirm which wallets need to be added?

divine-comedian commented 8 months ago

Yeah well I think this will also be changing soon the UX for connecting wallets https://github.com/Giveth/giveth-dapps-v2/issues/3370

and soon even more when the multisig login feature goes live in the next few weeks

and EVEN MORE will also change again when we add Solana to the dapp perhaps in about a month and a half

Big changes CURRENTLY are that we removed support for torus wallets (and users should import their private keys to a new compatible wallet like metamask) and we added in mobile support for MetaMask on mobile browsers such as safari or chrome

Right now the dapp is optimized for use with Metamask but you still should be able to connect with any browser desktop extension, mobile wallet apps are a bit more tricky

oyealmond commented 8 months ago

should we update this now @divine-comedian? cc @franculio

oyealmond commented 4 months ago

Hello @divine-comedian, I will move this issue to Notion. Is there something else we should consider to update documentation about wallets? Or this is it.

ahmadabugosh commented 4 months ago

@oyealmond I would add 1) Now that Solana is live, we should make it clear that those are counted as two different wallets atm (and they aren't connected) 2) Also mention the new chains we recently added like Polygon, Arbitrum etc.

divine-comedian commented 4 months ago

I moved this to the giveth-docs repo - good point @ahmadabugosh we don't have any docs for connecting with Solana at all.

You can keep this in Notion and reference this github issue @oyealmond