Update Kwenta UI to support L2 - Githubissues

Synthetixio / kwenta

DEPRECATED. Please use: https://github.com/Kwenta/kwenta

MIT License

11 stars 20 forks source link

Update Kwenta UI to support L2 #292

Open gberg1 opened 3 years ago

gberg1 commented 3 years ago

Summary

Dev Tasks

[x] Replace logo to support L2
[x] Use Js mono repo to support tx notifications
[x] Strip out (deprecated) notify.js
[x] Add network color indicators for Optimistic Mainnet and Kovan
[x] Change switch network toggle to an 'add L2 netork' button

QA Tasks

[x] On L2 the we display the kwenta L2 logo
[x] On L1 and L2 we have the new tx notification system that pops up on the bottom right @JChiaramonte7 can you explain how to run a transaction so that the notification pops up?
[x] On L2 the network color dot in the top right is gold
[x] On L1 there is a button that says 'Switch to L2'

Test link: https://kwenta-i6l0rgrnr-synthetixio.vercel.app/

@gberg1 found 2 bugs:

1) The success message on transaction completion says "Your transaction has successful" should say "Your transaction was successful"

2) Need to reproduce again but when trying to exchange synths on L2 it prompted for gas and to pay 1.5 ETH