timharding31 / stock-fox

StockFox is a clone of the popular stock-trading application, Robinhood
2 stars 1 forks source link

FSP To-Do List #1

Open mwmadsen67 opened 4 years ago

mwmadsen67 commented 4 years ago

Wiki Page Home


Database Schema


Sample State


MVP List


Backend Routes


Frontend Routes

mwmadsen67 commented 4 years ago

Hi Tim, docs are looking pretty good. I will break down my feedback per section:

Schema: Formatting is largely good, however underneath each table I am looking for a line about where there are indexes and what tables foreign keys might be pointing to. Check out the bluebird example to see what I'm talking about. Also, users should keep track of buying power. Stocks and Cryptos should have prices and company names. watch_lists should be a table called watched_stocks or watches, more fittingly, since there will only be one list of watched items. This table should have a user_id and a stock_id. I'm a little confused about what asset lists is supposed to be, it looks like it has polymorphic columns, you may have to explain to me what the idea of that is. You should also add a table that keeps track of which stocks the user owns (although looking back I see that transactions is a bonus mvp).

State Shape: The state shape should be formatted with javascript tags to make it look nicer. I would also try to fill out this state with more information from your backend, considering that each table should have its own slice of state. So think about the suggestions I made for the schema and then bring that up to the state.

MVP-List: Looks good.

Overall looking pretty good Tim, let me know if you have any questions.

mwmadsen67 commented 4 years ago

Hey Tim, just going to leave some comments on your backend and frontend routes.

Backend routes: the GET api/users/:id is not a necessary route, and it says something about chirps. You will probably want routes for a stock/crypto index so that you can get a lot of info at a time. Also if you plan on saving any stock info to your backend you will also need a POST route for those two tables. Routes for watches does not need to be nested under user since everything on robinhood only relates to the current user. Since you'll always have access to current user you will not need to nest user_ids. What are collections?

Frontend Routes: Ok I see that collections are watchlists. I would try to keep the naming consistent across your app. My only feedback for the frontend routes is to show which components have IndexItems.