version 1#3727
Conversation
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your solution correctly implements all functional requirements for the Person component. The three person objects are rendered with proper destructuring, optional age handling, gender-based partner terminology, and correct marriage status messaging. The only issue noted is a minor formatting style concern regarding the ternary operator layout in src/components/Person/Person.jsx — this falls under code style preferences rather than a functional blocker, and such formatting guidelines are typically handled by linters in the development workflow. Great work on meeting the core requirements!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| {person.age && <p className="Person__age">I am {person.age}</p>} | ||
| {person.isMarried ? ( | ||
| <p className="Person__partner"> | ||
| {person.partnerName} is my {person.sex === 'm' ? 'wife' : 'husband'} |
There was a problem hiding this comment.
This violates checklist item #2: Format ternary operator operands correctly - move each operand to the separate line. The ternary {person.sex === 'm' ? 'wife' : 'husband'} should be formatted across multiple lines for readability.
2 participants
https://dianakomar.github.io/react_person/