State

[simcha-c]

The state looks great! Couple of things...

• I think the user slice should have the portfolioIds array. It would make it easier to get the information when wanting to display the portfolio.
• Also, the portfolio slice doesnt need the userId since we're only going to show the portfolio of the current_user.

Otherwise, everything looks good!

Really good job!
Simcha

[davidrichard23]

So I feel that there might be a misunderstanding here. I was thinking that a single portfolios entry would be a single stock that the user has added to their portfolio--not a single entry containing all the data about each stock in their portfolio. So basically, each user would have one portfolios entry for each stock that they own. Maybe the name is a little bit confusing. Maybe portfolio_items or portfolio_stocks would be a better fit?

[davidrichard23]

Oh, rereading your comment, maybe I misunderstood 😬