Update 03-forms-and-events.md#31
Update 03-forms-and-events.md#31robots4life wants to merge 1 commit intometeor:masterfrom robots4life:patch-2
Conversation
Inconsistency between the code in step https://blaze-tutorial.meteor.com/simple-todos/02-collections.html#2-3-Render-Tasks-Collection and https://blaze-tutorial.meteor.com/simple-todos/03-forms-and-events.html#3-4-Add-Submit-Listener. ```javascript return TasksCollection.find({}); ``` is suddenly changed to ```javascript return TasksCollection.find({}, { sort: { createdAt: -1 } }); ``` Sure it makes sense to sort the results, however it might be better to have this sorting code only included later on, that is, in the next step where it is explained. https://blaze-tutorial.meteor.com/simple-todos/03-forms-and-events.html#3-5-Show-Newest-Tasks-First
|
It is described on line 101. Do you think that should be completely separate step? |
That would be less confusing and more helpful. With that also the opportunity could be taken to go into details about the The alternative to that could be to have line 101 moved up to line 70 and give a bit more detail about it. But then you have the discrepancy between the headline being I am under the impression a separate step could help beginners better, what do you think? |
Inconsistency between the code in step https://blaze-tutorial.meteor.com/simple-todos/02-collections.html#2-3-Render-Tasks-Collection and https://blaze-tutorial.meteor.com/simple-todos/03-forms-and-events.html#3-4-Add-Submit-Listener.
is suddenly changed to
Sure it makes sense to sort the results, however it might be better to have this sorting code only included later on, that is, in the next step where it is explained. https://blaze-tutorial.meteor.com/simple-todos/03-forms-and-events.html#3-5-Show-Newest-Tasks-First