Comments about reactrails-react-native-client

[justin808]

We should fetch the data in componentDidMount

https://facebook.github.io/react/docs/component-specs.html#mounting-componentdidmount

Because we don't want a race condition between render and loading. We also should have a spinner while we're waiting for the data to come back.

[badnorseman]

@alleycat-at-git

I was thinking adding these statements to CommentsContainer and let it control data to CommentsComponent. We would also need to initialize the action fetch data from Container rather than inside Component. What do you think?

 componentDidMount() {
    this.setState({
      dataSource: this.state.dataSource.cloneWithRows(this.props.customer.behaviors || [])
    })
  }
  componentWillReceiveProps(nextProps) {
    this.setState({
      dataSource: this.state.dataSource.cloneWithRows(nextProps.customer.behaviors || [])
    })
  }

I used different approaches for ListView, but the datasource definition is usually only set once in the component. I tend to define datasource in state.

this. state = { datasource: new ListView.DataSource({ rowHasChanged: (r1, r2) => r1 !== r2 }) }

When I use it as shown above.

Beforehand, I had this pattern in the Component, but I am thinking that it could be in Container instead and remove state from the component.