1a2b3c4dBobAn/Cyber-Wallet

PA Review: MVP 2: Stock Show Page MVP 3: Transaction module & Watchlist module & Search module

Opened this issue · 2 comments

1a2b3c4dBobAn commented
  1. Stock Page
  • Users can check individual stock page,
  • interact with live stock chart
  1. Transaction module & Watchlist module & Search module
  • Users can buy or sell stocks
  • Users can add stocks to watchlist or remove from watchlist
  • Users can search stock with stock name or symbol
1a2b3c4dBobAn commented

@niartenyaw Hi Aaron, take a look at the new live demo!

niartenyaw commented

Awesome work!

Needed

  • there needs to be some way to get back to the home dashboard from the stock pages (probably a logo in the navbar
  • If i add too many items to the watch list, it breaks. Try having a max-height and then make any overflow scrollable within the component

Nice to have, eventually

  • The charts are barely fitting the uppermost point, which is cutting off the glowing effect. Might want to give the chart a larger max value than your max data value
  • The pie charts are awesome, but it would be great to have an actual index of the holdings and the cash allocation listed somewhere else as well.
  • It would be great to have a loading screen for the chart when new info is requested (maybe a spinner over the current chart). It can be confusing to wait for a few seconds without a response.