SwaprHQ / swapr-dapp

DEX meta agregator
https://swapr.eth.limo/
GNU General Public License v3.0
49 stars 57 forks source link

[1739][FEATURE] - Update bridge modal colors and content #1820

Closed ElRodrigote closed 1 year ago

ElRodrigote commented 1 year ago

Fixes: #1739

Description

image image

How to test the changes

1) Pull this branch 2) run the project locally 3) Go to swapr bridge: baseUrl/#/bridge (depending on your localhost config, the base URL may change) 4) Connect your wallet and try to bridge assets from one chain to another 4.a) selecting Socket bridge will trigger an optional warning text in the bridge disclaimer modal 4.b) selecting LiFi bridge will display the regular disclaimer text

netlify[bot] commented 1 year ago

Deploy Preview for swapr ready!

Name Link
Latest commit 7d39da943d125a2d05b4fb8e02a897a282c7c8bf
Latest deploy log https://app.netlify.com/sites/swapr/deploys/64632029a526940008a107ac
Deploy Preview https://deploy-preview-1820--swapr.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 settings.

berteotti commented 1 year ago

Wait until @0xVenky approves the PR too

0xVenky commented 1 year ago

Hey, there is an extra line in Socket that says Socket asks for a signature and giving control of wallet. Can we remove that? And we are good to merge.

Its specifically shown for Socket and not others. Would be good to just show same warning for users.