Setup has_many between user and notification
refactor notifications_controller to ensure actions are scoped to user
refactor notifications_controller routes to be more conventional
refactor notifications routes to be fully nested under user
This is better rest architecture
Refactor view to move complexity to helper