ProductList code review

[osminosm]

note : usually this isn't done as an issue, these remarks should be done before merging from your feature branch to the develop branch, but until we have that, i'll post remarks as issues.

here you shouldnt provide the previous state when updating the state, the following would be sufficient: this.setState({refresh:true});

here the variable count is used to init the state a single time when its 0 then it gets incremented to 1, i guess there is no use in using this whole variable as you can initialize your state for the first time in the constructor like this :

constructor(props){
  super(props);
  this.state={
    allProducts: this.props.products,
    ... // rest of default state
  }
}

and you can remove the componentDidUpdate logic.

here Its useless to pass the products as parameter, since the function has access to this.state.currentProducts, remove the count conditional, and perhaps it should be a separate stateless component, not a method on ProductList.

here

return a descriptive message that there aere no products, instead of null;

remove unnecessary comment here

and think about defining propTypes for all the compoents