General Notes Client Side - React

[Nizar7zak]

shoestore/client/src/App.jsx

Line 40 in b9c46b5

<NotFound />

Avoid console.log and commented Lines in your codes, you missed handle the login state, your users can add to cart without login. and the login button didn't work !

some issue i saw it in your code...

  componentDidMount() {
    const { id, buttons , changeCart} = this.props;
    if(typeof buttons === 'string'){
    const products = JSON.parse(localStorage.getItem('products'));
    const addedToCart = products.some(item => item.id === id);
    this.setState({ addedToCart });
    changeCart(products.length)

  }

why we use component did mount here? I advice you to read this essay about local storage and what is the max size we can store in Local storage, should pass this data from parent component is better than this way. we can pass all data we need as props,

componentDidMount we use it when we have a http request in this component.

In contrast you did a great work and applied the concept of State and how we handle it very good, moreover you separate your components VERY GOOD.

GREAT WORK.