Added the option to filter todo by Added and Pipelines

Moved if statements around in view

Closes #24127

We’re being kept up to date the counter data but we’re not using it.
The only thing which is not real if is the number of projects that the 
user read changes the number of todos can be stale for some time.

The counters will be sync just after the user receives a new todo or mark any as done

- As todos are created/updated inside the TodoService
we repopulate the cache just there for both pending/done todos
- Todos as mark as done from the TodosController we update cache
there too
- All the added methods are kept in the User class for cohesion

Based on feedback from !4502

This reverts commit 13e37a3e.

This reverts commit 3e991230.

# Conflicts:
#	app/models/project.rb

Signed-off-by: NAlfredo Sumaran <alfredo@gitlab.com>

When a build fails for a commit, create a todo for the author of the
merge request that commit is the HEAD of. If the commit isn't the HEAD
commit of any MR, don't do anything. If there already is a todo for that
user and MR, don't do anything.

Current limitations:
- This isn't configurable by project.
- The author of a merge request might not be the person who pushed the
  breaking commit.