feat: add Sum component - #4388
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 accepts props with correct default values, renders the proper output format, uses named export, and App.jsx contains all five required Sum instances. The only noted issue is a code style preference regarding template literal usage (checklist item #2) in Sum.jsx. Since your code produces the correct output and functions properly, this style guideline is a non-critical preference that doesn't prevent task completion. Consider updating the paragraph to use template literal interpolation like {Sum of ${a} and ${b} is ${a + b}} for improved readability, but this doesn't block your approval. Great work on meeting all core requirements!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| import { Sum } from './components/Sum/Sum'; | ||
|
|
There was a problem hiding this comment.
This violates checklist item #2: Use string interpolation inside tag content. The paragraph should use a template literal like {Sum of ${a} and ${b} is ${a + b}} instead of separate JSX expressions.
| <p> | ||
| Sum of {a} and {b} is {a + b} |
There was a problem hiding this comment.
According to checklist item #2, you should use string interpolation inside tag content. Use a template literal instead: {Sum of ${a} and ${b} is ${a + b}}
DEMO LINK