Develop #2929
Develop #2929
Conversation
There was a problem hiding this comment.
Your code is well-structured and meets the core requirements of the task! 🎉 The minor optimization regarding the use of the spread operator in setTodos doesn't impact functionality, so it's not a blocker. Keep up the great work, and consider this optimization for future improvements! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| user: usersFromServer.find(user => user.id === userId), | ||
| }; | ||
|
|
||
| setTodos([...todos, newTodo]); |
There was a problem hiding this comment.
You are using the spread operator to create a new array when adding a new todo. Since setTodos is a state update function, it is not necessary to create a new array with the spread operator. You can directly pass the new array to setTodos without spreading it. This is a minor optimization according to the checklist: 'If you are using a non-mutating array method, you don't need to create a copy of the array'.
DEMO LINK