Skip to content

Create Person component#3788

Open
AntonyShaga wants to merge 1 commit into
mate-academy:masterfrom
AntonyShaga:develop
Open

Create Person component#3788
AntonyShaga wants to merge 1 commit into
mate-academy:masterfrom
AntonyShaga:develop

Conversation

@AntonyShaga

Copy link
Copy Markdown

Implemented:

  • created Person component
  • passed the entire person object as a prop
  • rendered name, optional age and partner info
  • added correct partner role: wife / husband
  • added not married message
  • tests passed

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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 receives the full person object as a prop, destructures it properly, and handles optional age so that it’s omitted when not provided while still displaying it when present. The partner text is rendered correctly as wife for male and husband for female persons, and the unmarried case shows the required “I am not married” message. All required class names (Person, Person__name, Person__age, Person__partner) are used as specified, and the misha, olya, and alex constants remain unchanged. Your conditional logic is simple and readable, and the ternary for partner wording is formatted cleanly, matching the checklist expectations. Nice work meeting both the functional requirements and the style constraints for this task.


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

Footnotes

  1. Rate AI review example

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