- We can probably move this sort of functionality in the ApplicationController. We probably need to access the current user in other places besides this specific controller in the future.
- In the index controller action we are building up the query based off certain params being passed in. This query generation does not belong in the controller. We can use something like the interactor pattern to pull this logic out and generate our query for us, or the next best place would be in the model.
- I think a nice added feature to this do list is being able to do a bulk update on tasks that are completed. Since we are just using RoR here with no JavaScript we get a new page load everyt