Conversation
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your implementation is approved. All functional requirements are met: the Sum component correctly accepts props a and b, renders the expected output, uses default values of 0, and the App contains the required 5 instances. While the code uses separate JSX expressions instead of template literals for the output text (which differs from the preference expressed in checklist item 2), this is a style preference rather than a functional issue. The code works correctly and passes all tests. Consider updating the <p> content to use template literals like {`Sum of ${a} and ${b} is ${a + b}`} in future iterations, as this is more concise and is generally considered a best practice in React.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| import { Sum } from './components/Sum/Sum'; | ||
|
|
||
| export const App = () => ( | ||
| <> |
There was a problem hiding this comment.
This violates checklist item 2: Use string interpolation inside tag content. Replace with template literal: {Sum of ${a} and ${b} is ${c}}
| <p> | ||
| Sum of {a} and {b} is {c} |
There was a problem hiding this comment.
This violates checklist item 2: Use string interpolation inside tag content. Replace with template literal: {Sum of ${a} and ${b} is ${a + b}}
https://pavlo111.github.io/react_sum/