Not a huge fan of the `notification` name that this object is injected into.  It's not descriptive enough IMO when used throughout the application.

I'd prefer something a little more representative of the array of notifications that are being modified, such as using the name `notifications`.

Conversely I'd be fine with using `notificationService` as the name it's injected into as well, to make clear what that property does.