buildonviction / tomox-sdk-ui Goto Github PK

The portfolio overview is not present the percentage of the balance

Markets page is now "static" in each page reload.
We will integrate WebSocket API so it will automatically fetch new data without reloading

Please take a look at this picture (the green/red color).

Notice the proportion between 0.02 vs 0.17, 0.05 vs 0.17, ...

The last order total will be 100%, all other above orders are proportional to it.

It seems that only after the user is logged in, client code send API request to get pairs.

The correct logic should be: for Markets page, always get pairs even if user is logged in or not

On first load of Wallet page, the token balance component shows undefined

Only when hovering the mouse on it, the balance shows correctly.

It's similar to the order book on Kucoin exchange

This issue is caused by function ordersInitialized in file src/store/domains/orders.js (Redux)

If order amount has 3 digits (smaller than 1000), the value displayed is corrrect, but if order amount has 4 digits, it shows NaN.undefined 🤕

On the server, we removed Exchange smart contract, updated balance locally in MongoDB

Client code needs updates:

• Get balance from server API instead from the blockchain
• Remove feature allowance in Wallet page

We need to do this to avoid unnecessary requests that the server has to send

After input Price and Amount, the value in Amount auto change to a large number.

We should send websocket message from server after token is approved and handle that message on client to hide the Pending state.

It makes the UX more dynamic.

Open Orders section should be updated automatically (without reloading).

This happens in case user logged in with private key / keystore / mnemonic.
It doesn't happen with Metamask because when logging in with Metamask, user is redirected to Login page after reloading page.

We should redirect user to Login page in case user logged in with private key / keystore / mnemonic, so the UX will be similar to Metamask.

It is similar to the Markets section on Kucoin:

Token's balance should be updated after order match

Replace the current chart (using https://github.com/rrag/react-stockcharts) by TradingView.

Because TradingView is widely used by many exchanges so users are familiar with it.

We need to provider support for Ledger Nano S wallet.

This issue has low priority

Price token decimals inconsistent between tomox-sdk-ui and tomox.

• Send Order, Parse Order

Please make a change for TradingView chart, it's always using the UTC+0 timezone, we should be using local timezone.

We are TOMO, not ETH

When I unlock any tokens in Wallet window, the alert message will appear and the status only updated when refreshing the web page.

in ALL tab, there are some cancelled orders and open orders but when I click on CANCELLED tab, there is nothing there

As you can see in Redux, the time is 2019-05-08T14:45:14+07:00
But in the Open Orders section, it's 05-08 14:14:45

Cannot read property 'baseTokenDecimals' of undefined when loading Exchange page.