Implemented the Sum component and performed the action.#4265
Conversation
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your implementation correctly fulfills all functional requirements for the Sum component and App — all five required instances are present with proper default values. However, checklist item #2 is violated in src/components/Sum/Sum.jsx. The code currently uses separate JSX expressions inside the <p> tag instead of template literal interpolation. Change the paragraph content to use a template literal: <p>{Sum of ${a} and ${b} is ${a + b}}</p> instead of individual expressions. This is an explicit checklist requirement that must be corrected.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Important
Fix the issues, commit the changes, and then re-request my review using the Re-request review button2 in the Reviewers tab.
Tip
If you have any comments about this review, please leave a message in the Add a comment section below and mention me with the @mateacademy-ai-mentor tag3.
I will read your message during my next review of your pull request. Please do not reply directly to my comments, as that feature is not supported yet.
Caution
Do not close the pull request until the review is complete. Closing it early can lead to unexpected behavior and disrupt the review process.
Footnotes
| <p>Sum of 10 and 0 is 10</p> | ||
| <p>Sum of 0 and 5 is 5</p> | ||
| <p>Sum of 0 and 0 is 0</p> | ||
| <div> |
There was a problem hiding this comment.
This violates checklist item #2: Use string interpolation (template literals with backticks) inside tag content. The good example shows using {Sum of ${a} and ${b} is ${a + b}} instead of separate expressions.
| <p> | ||
| Sum of {a} and {b} is {c} |
There was a problem hiding this comment.
This violates checklist item #2: Use string interpolation (template literals with backticks) inside tag content. Use: <p>{Sum of ${a} and ${b} is ${c}}</p>
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your implementation is approved. All requirements are now satisfied: the Sum component correctly uses template literal interpolation for the paragraph text, implements named export with default prop values of 0, and all five required instances are present in App.jsx with proper props. Code style is consistent with the checklist requirements. The previous checklist item #2 issue has been successfully resolved. Great work!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
2 participants
https://ValyaMeln.github.io/react_sum/