Grouped email notifications

[bastianbeier]

Description

Allow grouping of multiple events in a single email.
Needs to be merged after #10838

Related Issue

#10793

Motivation and Context

How Has This Been Tested?

• test environment: localhost
• test case 1: Test if grouped email can be sent successful (trigger multiple events, run cli command, check mailbox)

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation ticket raised:

[update-docs]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

[bastianbeier]

TODO:

• tests
• documentation

[wkloucek]

what does weekly mean in this context? I'd expect notifications with a creation date within the "last 7 days" / last 168 hours!?

[bastianbeier]

daily and weekly are just groups. You can trigger the sending of a group using the notifications send-email --daily or notifications send-email --weekly cli command.
You could also trigger the weekly job every 4 days or every 8 days but you have to consider the TTL of the events in the store when triggering not that often.

[kobergj]

@bastianbeier @2403905 do we consider the sonarcloud warnings false positives?

[sonarqubecloud]

Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
30.0% Coverage on New Code
4.6% Duplication on New Code

See analysis details on SonarQube Cloud