Don't duplicate stuff
Closed this issue · 1 comments
malakada commented
var FilterableIssueList = React.createClass({ getInitialState: function() { return { displayClosed: false, issues: [], repo: 'andrewrk/groovebasin', }; },
is the same as:
var SearchBar = React.createClass({ getInitialState: function() { return { displayClosed: false, repo: 'andrewrk/groovebasin', }; },
This should be consolidated into one location.
malakada commented
So on a closer review, this is actually good/ok because it individually/locally needs to keep track of the state of the checkbox, and one component should not rely on another component for this information. Closing because this is actually correct.