0xProject / 0x-launch-kit-frontend

Apache License 2.0
114 stars 208 forks source link

Automatic UI Update Improvement #87

Closed Agupane closed 5 years ago

Agupane commented 5 years ago

Check if is necesary to use polling in order to receive the user account changed / networks events on wallets like nifty wallet. The current implementation uses events and it's supported on metamask but not tested on nitfy wallet.

Based on the comment: https://github.com/0xProject/0x-launch-kit-frontend/pull/84#discussion_r261593322

pablofullana commented 5 years ago

IMO, let's support just Metamask for now. I don't see any benefits in adding more code in this initial iteration just to support a wallet that's not as mainstream as MM, specially because of upcoming deadlines.