solution #2776
solution #2776
Conversation
There was a problem hiding this comment.
Great job on implementing the task requirements! 🎉 While there are a couple of issues that need attention, such as ensuring the Person component is correctly imported and adjusting how partner details are accessed, these are minor and don't significantly impact the core functionality. Keep up the good work, and consider these improvements for future projects! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| <Person person={misha} /> | ||
| <Person person={olya} /> | ||
| <Person person={alex} /> |
There was a problem hiding this comment.
The Person component is used here but is not imported or defined in this file. Make sure to import the Person component from its respective file or define it within this file to avoid runtime errors.
src/components/Person/Person.jsx
Outdated
| <p className="Person__partner"> | ||
| {person.isMarried ? | ||
| `${person.partner.name} is my | ||
| ${person.partner.sex === 'f' ? | ||
| "wife": | ||
| "husband"}` : | ||
| "I am not married"} |
There was a problem hiding this comment.
The person.partner.name and person.partner.sex properties are accessed here, but the person object does not have a partner property in the current structure. You should access person.partnerName and use a condition to determine the partner's sex based on the person.sex property.
DEMO LINK