@fazzaamiarso
Posted
Hi Eileen! Great Work!
I have a few suggestions for you.
- When you don't want to use a the returned array, don't use
.map
, but use.foreach
. I have made an improvements, but using reduce.
useEffect(() => {
const counts = data.reduce((acc, currEl) => user.read ? acc + 1 : acc ,0 )
setCount(counts)
}, [])
- For your conditional styling, I think it's better to add/remove classes conditionally rather than
style
props or use css modules.
I hope it helps! Cheers!