Solution #3541
Solution #3541
Conversation
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Excellent work! Your solution meets all the functional requirements, and the components are well-structured. I am approving this submission.
There is one minor code style improvement I'd like to point out for future reference. In src/components/Person/Person.jsx, the ternary operator should be formatted across multiple lines to improve readability, as per [CHECKLIST ITEM #2].
For example:
const partnerLabel = sex === 'm'
? 'wife'
: 'husband';This is a small detail, but it makes the code easier to read. Keep up the great job!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| // export const Person = ({ person }) => (); | ||
| export const Person = ({ person }) => { | ||
| const { name, age, sex, isMarried, partnerName } = person; | ||
| const partnerLabel = 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:'. For better readability, please format this ternary operator across multiple lines.
2 participants
No description provided.