React Sum Props - #4396
Conversation
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your implementation correctly fulfills all functional requirements - the Sum component works as expected, handles missing props with defaults, uses named export, and all required Sum instances are present in App.jsx. All tests pass. The only issue is a code style preference regarding string interpolation syntax in checklist item #2, which doesn't affect functionality. This is a minor style difference rather than a functional blocker, and your code produces the correct output.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| @@ -1,14 +1,17 @@ | |||
| import React from 'react'; | |||
| import './App.scss'; | |||
| import { Sum } from './components/Sum/Sum'; | |||
There was a problem hiding this comment.
This violates checklist item #2 which requires string interpolation inside tag content. Use template literal syntax: {Sum of ${a} and ${b} is ${a + b}}
| <p> | ||
| Sum of {a} and {b} is {a + b} |
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}}
DEMO LINK