need to add UserInfo class #2927
Conversation
There was a problem hiding this comment.
Great job on implementing the task! 🎉 While there are a few minor issues like typos in function names and accessibility improvements, they don't significantly impact the functionality. Keep up the good work and consider addressing these minor points for even cleaner code in the future! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| <span className="error">Please choose a user</span> | ||
| </div> | ||
| const [usersList] = useState<User[]>(usersFromServer); | ||
| const [todoList, setuTodoList] = useState<Todo[]>(todosFromServer); |
There was a problem hiding this comment.
The function name setuTodoList seems to be a typo. It should likely be setTodoList to match the naming convention and improve code readability.
| const [titleError, setTitleError] = useState(false); | ||
| const [userError, setUserError] = useState(false); | ||
|
|
||
| function onSubmitChacge(event: React.FormEvent) { |
There was a problem hiding this comment.
The function name onSubmitChacge seems to be a typo. It should likely be onSubmitChange to match the intended functionality.
| </div> | ||
|
|
||
| <div className="field"> | ||
| <label htmlFor=""> |
There was a problem hiding this comment.
The htmlFor attribute in the label should match the id of the select element for better accessibility. Consider adding an id to the select element and updating this attribute accordingly.
DEMO LINK