Implement Person component#3793
Conversation
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your solution is approved because it correctly implements all required behaviors for the Person component and its usage in App. The Person component is used three times with the full person object passed as person, and age is rendered only when it exists, which matches the optional-age requirement. The marital status logic is handled correctly: unmarried people show "I am not married", and married people refer to their partner as wife or husband based on the sex field.
You also followed the checklist items well: props are properly destructured, ternary expressions are formatted with operands on separate lines, and your conditional rendering avoids mixing several cases into one expression. There are no critical or blocking issues in the current implementation, and the code is clear and easy to read. Nicely done keeping the data objects unchanged and matching the specified class names throughout the components.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
2 participants
Hello!
DEMO LINK