Skip to content
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

solution #1413

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

solution #1413

wants to merge 2 commits into from

Conversation

rokster112
Copy link

Copy link

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job on your implementation! 🎉 You've successfully split the App into multiple components and handled styles separation, which aligns well with the task requirements. The use of key props in your PostList and CommentList components is correctly implemented, ensuring proper rendering and performance. Keep up the excellent work, and continue refining your skills! 😊

✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨

src/App.jsx Outdated
return (
<section className="App">
<h1 className="App__title">Static list of posts</h1>
<PostList posts={posts}/>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PostList component should have a key prop assigned to the root element inside the iteration. According to the checklist, you need to add a key to the root tag inside an iteration (the first tag after the .map call). Ensure that each PostInfo component in the PostList has a unique key prop, such as key={item.id}.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants