- first of all, trim the input, currently I can add an empty task. - then, when you toggle the item, you are mutating state directly : if (element.id === item.id) { element.completed = !element.completed; } instead, return a new object with the opposite value, something like this: if (element.id === item.id) { return {...element, completed: !element.completed}; } - !!! also, big bug: try to add 2 tasks with the same name and delete one of them, then see what happens

@Prudhvi Raj Maddukuri that doesn't mean you can ignore the rest of the code, I was expecting otherwise from you 🙃

what's up bro, hope you are good. overall great job, here are some comments though: - use arrow functions instead of functions declarations. it's pretty much a standard. just take a look at this example: function InputTextField(prop: { placeholder: string }) { return <input type="text" placeholder={prop.placeholder} />; } vs const InputTextField = ({ placeholder: string }) => <input type="text" placeholder={prop.placeholder} />; idk, maybe it's the way I am used to do things, but it's mode readable react wise if that makes sense 😅 - you can define this function: const composeClassName = (text: string) => text.toLowerCase().split(' ').join('-') + '-button'; inside the component you are using it, since it's a one time use function - in react, when you use map, you need to add a key attribute to the list elements. this is pretty important, look into it

@Prudhvi Raj Maddukuri I'm not forced to use AI, but my boss recommends me to use it, and I use it (chatgpt), but only because I started to understand the code, first few months I barely used it. And regarding the key, you should always use it with map, regardless of the fact that the list modifies its order or not. Just pass the value to it

Merry Christmas!

Yo bro, the url is not working, it requires access. Share the public one

@Cade Warbington this is it 👌

This is a lvl 7 kata for real ? I don't understand a thing 🥲😅