Develop #1450
Develop #1450
Conversation
There was a problem hiding this comment.
Great job on implementing the core functionality! 🎉 While there are a few minor issues, such as a typo in comments.length and a variable naming suggestion, these don't critically impact the functionality. Keep up the good work and consider these improvements for future projects! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| <div className="PostInfo__header"> | ||
| <h3 className="PostInfo__title">doloremque illum aliquid sunt</h3> | ||
| function getUserById(userId) { | ||
| const foundedUser = usersFromServer.find(user => userId === user.id); |
There was a problem hiding this comment.
The variable name foundedUser is not semantically correct. A more appropriate name would be foundUser, as it indicates the user that was found.
src/components/PostInfo/PostInfo.jsx
Outdated
|
|
||
| <hr /> | ||
|
|
||
| {comments.lenght !== 0 ? ( |
There was a problem hiding this comment.
There is a typo in comments.lenght !== 0. It should be comments.length !== 0. This typo will cause an error because lenght is not a valid property of an array.
DEMO LINK