Follow-up: Integrate multiple notification objects

@Grevil and @Anybody will hopefully implement this asap. Busy times... Until then, this can be used with just one entry like before.

I just implemented a solution that works fine without AJAX on the second branch. This could be our go-to solution if we keep struggling with it. I think that with this simpler solution we could actually transform the "remove" checkbox into a button as well perhaps, if we could implement it the same way as the add button.

The solution without AJAX is ready, working fine and tested! @Anybody, are you taking a look?

Anybody → closed merge request !30

Anybody → closed merge request !31

Anybody → reopened merge request !31

@LRWebks thanks, I left some final comments. We could have some more tests to be even more sure it works as expected:

• Remove all notification object entries (this should also have a mail test as we have the new case that the notifications objects can be empty in config now)
• Remove multiple notification objects at once

Could you add these finally? Should be simple, I think.

Good enough, thank you :)

Automatically closed - issue fixed for 2 weeks with no activity.