add task solution - #4380
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 handles all core functionality - the Sum component accepts a and b props with proper defaults, renders the correct message format, and the App component contains all five required instances. However, there is an explicit requirement to use named export only (not default), and your Sum.jsx file currently includes export default Sum;. This line must be removed to comply with the checklist requirement [CHECKLIST ITEM: use named export (not default)]. The named export export const Sum is already present, so you simply need to delete the default export line.
✨ 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 ${a} and ${b} is ${a + b}`}</p> | ||
| ); | ||
|
|
||
| export default Sum; |
There was a problem hiding this comment.
This line is unnecessary. The task requires using named export only, not default export. Remove this line to follow the requirement: 'use named export (not default)'.
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your implementation is complete and correct. The Sum component uses named export only (no default export), properly accepts a and b props with default values of 0, renders the correct paragraph format using string interpolation, and the App component contains all five required Sum instances with proper props. Code style requirements are satisfied with components under 3 attributes on single lines.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
2 participants
DEMO LINK