Conversation
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Excellent work! Your solution correctly implements all the functional requirements. The Sum component works as expected with both provided and default props, and the App component renders all test cases perfectly.
I'm approving your submission, but I have two small suggestions for future improvements. First, in src/components/Sum/Sum.jsx, you can use a single template literal like {`Sum of ${a} and ${b} is ${a + b}`} to construct the string, which is a bit cleaner and aligns with the checklist [CHECKLIST ITEM #2]. Second, the linter failed because of the file extension in your import statement in src/App.jsx; it's best to write it as import Sum from './components/Sum/Sum';.
These are minor style points and don't affect the functionality. Great job overall!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| export const Sum = ({ a = 0, b = 0 }) => ( | ||
| <p> | ||
| Sum of {a} and {b} is {a + b} | ||
| </p> |
There was a problem hiding this comment.
This violates checklist item #2: 'Use string interpolation inside tag content'. Using a template literal like {`Sum of ${a} and ${b} is ${a + b}`} would make the string construction clearer and match the example provided in the checklist.
https://vakt159.github.io/react_sum/