-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PR for Creating investing page on crypto site #158
Conversation
e6cc10f
to
fc7f909
Compare
pages/invest/index.js
Outdated
} | ||
.shoppinglist-container { | ||
display: flex; | ||
flex-direction: row; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NIT: Not needed, by default it takes the direction as row
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure!!👍
pages/invest/index.js
Outdated
{stock.map((itemName) => { | ||
return <Card key={itemName} stock={stockData[itemName]} />; | ||
})} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since you are returning only 1 line of code, you can transform this like this
{stock.map((itemName) => <Card key={itemName} stock={stockData[itemName]} />)}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will do the changes. Thanks for the review😀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May consider below to please the human eyes
{ stock.map( itemName => <Card key = {itemName} stock = { stockData[itemName] } />) }
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FTL Rocks! ⚡
Create an investing page where users can buys and sell stocks. Fixes #72
The input fields
stock Name
andListed Price
are prefilled when clicked onBUY
/SELL
buttons.Subsequent PRs - #186 - Adds validations for user login, wallet and enough money