telosnetwork / telos-bridge

The Telos LayerZero bridge enables seamless asset transfers between Telos and other blockchains
https://bridge.telos.net
5 stars 3 forks source link

#19 | Handle wallet deeplinks on mobile #21

Closed ezra-sg closed 10 months ago

ezra-sg commented 10 months ago

Fixes #19

Description

This PR updates how the app handles login links, especially on mobile. This includes:

  1. removing brave altogether, to be fixed in #20
  2. on mobile, changing login button language to 'Continue on [wallet name]' and opening the selected wallet app when tapped
  3. hiding Core wallet entirely on mobile, as they have no deeplink capability

note: at the time of writing, the L0 servers are responding with 500 for the wallet icons. this is happening in prod too, it is not related to this PR

Test scenarios

Checklist:

netlify[bot] commented 10 months ago

Deploy Preview for telos-bridge ready!

Name Link
Latest commit 66936194d62dd270212646379fb65b27c47fa658
Latest deploy log https://app.netlify.com/sites/telos-bridge/deploys/65400aa25ca2cb0008c25073
Deploy Preview https://deploy-preview-21--telos-bridge.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.