Posts Not solution #1457
Posts Not solution #1457
Conversation
There was a problem hiding this comment.
Hey there! 👋 Thanks for the submission! There's just one small tweak needed. In CommentList.jsx, make sure the key prop is on the <CommentInfo> component, not the div inside it. This ensures React can efficiently update the list. You're almost there! 🎉
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
🔄 Fix the issues, commit the changes, and then re-request my review using the Re-request review button in the Reviewers tab.
🤖 If you have any comments about this review, please leave a message in the Add a comment field below and mention me with @mateacademy-ai-mentor tag. I will read your message when I perform the next review of your pull request.
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
[CODE: 7]
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
[CODE: 7]
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
[CODE: 7]
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
[CODE: 7]
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
src/App.jsx
Outdated
| </div> | ||
| function getCommentsByID(postID) { | ||
| return ( | ||
| commentsFromServer.filter(comment => comment.postId === postID) || null |
There was a problem hiding this comment.
| commentsFromServer.filter(comment => comment.postId === postID) || null | |
| commentsFromServer.filter(comment => comment.postId === postID) |
filter always returns an array
src/components/PostInfo/PostInfo.jsx
Outdated
| <p> | ||
| {' Posted by '} | ||
|
|
||
| <UserInfo user={post.user} /> |
There was a problem hiding this comment.
| <UserInfo user={post.user} /> | |
| {post.user && <UserInfo user={post.user} />} |
src/components/UserInfo/UserInfo.jsx
Outdated
|
|
||
| export const UserInfo = ({ user }) => ( | ||
| <a className="UserInfo" href={`mailto:${user?.email}`}> | ||
| {/* User is properly passed, checked via console.log(user) */} |
There was a problem hiding this comment.
| {/* User is properly passed, checked via console.log(user) */} |
-DEMO LINK