If a user visits the site with a non-web3-enabled browser on DESKTOP, we should ask the user to install MetaMask or Brave, instead of Trust Wallet

DoD:

• user sees metamask on desktop
• user still see trust on mobile

1rem border-radius for token icons that load as squares.

See below
https://cl.ly/de0c73b062fa

• calculateInput
• calculateOutput

if reverse balances are zero, user can specify both inputs

blocked by #26, #27, #28

DoD

• After use confirm tx, the form should be cleared and return to empty state (clear input, output, inputCurrency, outputCurrency, and lastEditedField)

Drizzle should be updating the balance automatically when Transfer event is fired. However, the event is getting fired, but balance is not updating.

• investigate why drizzle isn't updating
• we might have to manually handle the event to update ERC20 balance

Calculate-Input-given-exact-output

use big number

Token balance should display 0 instead of NaN

Documentation

https://hackmd.io/hthz9hXKQmSyXfMbPsut1g?view#ethToTokenTransferInput

Notes

• Refer to pattern used in Swap screen. Start with onClick and exchange-utils

DoD

• If I select a ETH-to-ERC20 pair and specify input on the Send screen, the UI should invoke ethToTokenTransferInput on the exchange contract
• verify in the etherscan that the correct amount is transferred

Minimum-and-Maximum-values

My eyes were not drawn to it.

Cipher browser was acquired by Coinbase

Cipher deeplink should be replaced with coinbase wallet
Coinbase Wallet by Toshi https://itunes.apple.com/us/app/coinbase-wallet/id1278383455?mt=8

background: #E5E5E5;

DoD:

• Implement inactive state for CTA Button
• hook up inactive state to all form. If form is invalid, CTA should be inactive

Refer to implementation in swap page - start from componentWillReceiveProps

DoD:

• when i enter input, output should be updated
• when i enter output, input should be updated (verfy with the reverse of input > output)

The above should work for:

• ETH to ERC20 Swap
• ERC20 to ETH Swap
• ERC20 to ERC20 Swap

• (estimated) should go to input if user type in output
• / should redirect to swap and make button selected

https://cl.ly/140i2H1S1m0b

Entered 0.1 ETH and now not sure what to click.

Documentation

Specs

Summary

• In order for a user to perform any ERC20 to X swap, the cooresponding exchange must already have approval to move tokens on behalf of the user

DoD

1. When a token is selected from CurrencyInputPanel and the corresponding exchange's approval amount for the user is lower than the user's specified input, we should show:

2. The CTA buttom should also be in an inactive state if exchange does not have approval.

3. The user should be able to click on unlock create an ApprovalTx for decimals * 10^8 (ping @chikeichan for more details)

4. if you use drizzle's cacheSend method to make the ApprovalTx, the corresponding tx should be automatically stored and watched by the drizzle store. You should be able to use that state to render this:

5. When Tx is confirmed, the UI should be updated to allow trading.

6. You should be able to use something like redux-localstorage to store the drizzle transactions and transactionStack state in localstorage, and rehydrate when the app refresh. The goal is to store pending tx hash in localstorage, so that when user refresh and come back to the site later, the pending state will still persist. We should only store what's necessary, meaning that we should only keep transactions and transactionStack instead of the entire redux state.

Do trades get matched in single-price batches?
or is this a continuous markets?

Important because continuous markets are not secure against front running.

https://cl.ly/1X070o3s2B2o

• See that OMG is selected
• See that the OMG token row is not highlighted

DoD:

• If a token is selected, we should show the row with BLACK text and $mercury-gray background-color

border: 0.5px solid #FF8368;

DoD:

• implement error state
• hook up to all CurrencyInputPanel

Calculate-Output-given-exact-input

USE BIG NUMBER

Calculate-Input-user-specifies-output

• use bignumber
• verify answer should equal 1 / calculateOutput()

Currently exchange rate is calculated by a formula when user change input

DoD:

• We should watch the contract's ETH and token balance and update exchange rate on UI as they change

Exchange addresses here are all outdated and won't work:
https://github.com/Uniswap/uniswap-frontend/blob/master/src/ducks/addresses.js

Token addresses should be the same. Here are the new exchange addresses on Rinkeby:

BAT
Token: 0xDA5B056Cfb861282B4b59d29c9B395bcC238D29B
Exchange: 0x9B913956036a3462330B0642B20D3879ce68b450

DAI
Token: 0x2448eE2641d78CC42D7AD76498917359D961A783
Exchange: 0x77dB9C915809e7BE439D2AB21032B1b8B58F6891

MKR
Token: 0xF9bA5210F91D0474bd1e1DcDAeC4C58E359AaD85
Exchange: 0x93bB63aFe1E0180d0eF100D774B473034fd60C36

OMG
Token: 0x879884c3C46A24f56089f3bBbe4d5e38dB5788C0
Exchange: 0x26C226EBb6104676E593F8A070aD6f25cDa60F8D

ZRX
Token: 0xF22e3F33768354c9805d046af3C0926f27741B43
Exchange: 0xaBD44a1D1b9Fb0F39fE1D1ee6b1e2a14916D067D

DoD:
When user paste token address of a token and the token is not on our dropdown:

• get all token data from drizzle
• show a spinner in the dropdown while data is loading
• when data is loaded, display a new row with the token's symbol and logo
• if user click on the row, it will select the token
• token balance should be reflected accurately

Error state for when a user is connected to web3, signed into metamask, but NOT on ethereum mainet. ie when they're on a testnet