https://durmus-w3-ecommerce.netlify.app/

[mustafadurmusNl]

No description provided.

[tlorent]

Great work @mustafadurmusNl ! 👏 Let's go for the final project.

[tlorent]

What do you mean with 'since categories are strings'? 🙂 Why could categories not be used for the key? As long as they are unique, static values among siblings (so each li has a different value) it is fine.

[mustafadurmusNl]

I thought key should be a numeric value not a string. And in my studies i observed that  index is  best practice as unique key unless the server contains a unique Id   Sent from my iPhoneOn 29 Sep 2024, at 17:50, Tim Lorent ***@***.***> wrote: @tlorent approved this pull request. Great work @mustafadurmusNl ! 👏 Let's go for the final project. In week3/project/ecommerce/src/CategoryList.jsx:

@@ -0,0 +1,23 @@

+/* eslint-disable react/prop-types */ +const CategoryList = ({ categories, selectedCategory, handleCategoryChange }) => { + if (!Array.isArray(categories)) { + return <div>Error: Categories data is not an array.</div>; + } + + return ( + <div className="categories"> + {categories.map((category, index) => ( + <div + key={index} // Use index as the key since categories are strings What do you mean with 'since categories are strings'? 🙂 Why could categories not be used for the key? As long as they are unique, static values among siblings (so each li has a different value) it is fine. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>