Skip to content

Commit

Permalink
Make visibleTodos a required property
Browse files Browse the repository at this point in the history
Since TodoList's todos property is required, visibleTodos should be
required too.
  • Loading branch information
mjethani committed Nov 30, 2015
1 parent 42e7afd commit 09eaf64
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docs/basics/ExampleTodoList.md
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ App.propTypes = {
visibleTodos: PropTypes.arrayOf(PropTypes.shape({
text: PropTypes.string.isRequired,
completed: PropTypes.bool.isRequired
})),
}).isRequired).isRequired,
visibilityFilter: PropTypes.oneOf([
'SHOW_ALL',
'SHOW_COMPLETED',
Expand Down
2 changes: 1 addition & 1 deletion docs/basics/UsageWithReact.md
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,7 @@ App.propTypes = {
visibleTodos: PropTypes.arrayOf(PropTypes.shape({
text: PropTypes.string.isRequired,
completed: PropTypes.bool.isRequired
})),
}).isRequired).isRequired,
visibilityFilter: PropTypes.oneOf([
'SHOW_ALL',
'SHOW_COMPLETED',
Expand Down

0 comments on commit 09eaf64

Please sign in to comment.