🐛 Bug: Summary mail may contains notifications that should not be in the summary mail

[Skylsmoi]

Tracim version

4.8.0

Issue description

The summary mail received, at most, every day, should contain only unread notifications and mentions from the last 24h.

It sometimes contains notification or mentions that have already been read.

Todo:
Add details about which read notifications can be in the summary mail

How to reproduce

Todo

Screenshots

No response

What browsers are you seeing the problem on?

No response

What devices are you seeing the problem on?

No response

Related PR

No response

[Skylsmoi]

Update:
It doesn't contains notification already read. But it lacks filtering some notification that are filtered in the notification wall. This makes it looks like it counts read notification.

A beginning of the fix has been made here ed7ce90
It fixes:

• remove the events where the user is the author
• remove the mentions because they are counted in the specific mention count
• remove the entity type WORKSPACE to better match the notification wall. Notification wall actually shows the created spaces but if the space is "on demand", we don't want to count the notif of space we are not part of in the summary mail