Skip to content

Commit 612e9d1

Browse files
clean code after using flux style
1 parent c350cba commit 612e9d1

File tree

3 files changed

+3
-14
lines changed

3 files changed

+3
-14
lines changed

ep24-marching-towards-flux/app/components/App.jsx

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,6 @@ export default class App extends React.Component {
3333
.then( (responseData) => this.setState({todos: responseData.todos} ));
3434
}
3535

36-
handleDelete (idToBeDeleted) {
37-
api.deleteTodo(idToBeDeleted)
38-
.then( () => { return api.getTodos() })
39-
.then( (responseData) => this.setState({todos: responseData.todos} ));
40-
}
41-
4236
handleSubmit (event) {
4337
event.preventDefault();
4438

@@ -73,7 +67,6 @@ export default class App extends React.Component {
7367
</form>
7468

7569
<DisplayList
76-
handleDelete={this.handleDelete.bind(this)}
7770
todos={this.state.todos} />
7871

7972
<footer>

ep24-marching-towards-flux/app/components/DisplayItem.jsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ export default class DisplayItem extends React.Component {
3838

3939
handleDeleteTodoClick (todo) {
4040
TodoActions.deleteTodo(todo);
41-
//this.props.handleDelete(todo.id);
4241
}
4342

4443
render () {
@@ -83,6 +82,5 @@ export default class DisplayItem extends React.Component {
8382
}
8483

8584
DisplayItem.propTypes = {
86-
todo: React.PropTypes.object.isRequired,
87-
handleDelete: React.PropTypes.func.isRequired
85+
todo: React.PropTypes.object.isRequired
8886
}

ep24-marching-towards-flux/app/components/DisplayList.jsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,7 @@ export default class DisplayList extends React.Component {
88
{ this.props.todos.map((todo, i) => {
99
return <section id="main" key={todo.id}>
1010
<DisplayItem
11-
todo={todo}
12-
handleDelete={this.props.handleDelete} />
11+
todo={todo} />
1312
</section>
1413
}) }
1514
</ul>
@@ -18,6 +17,5 @@ export default class DisplayList extends React.Component {
1817
}
1918

2019
DisplayList.propTypes = {
21-
todos: React.PropTypes.array.isRequired,
22-
handleDelete: React.PropTypes.func.isRequired
20+
todos: React.PropTypes.array.isRequired
2321
}

0 commit comments

Comments
 (0)