-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(caldav): Only track interactions of event organizers #38542
Conversation
/backport to stable27 |
/backport to stable26 |
/backport to stable25 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also limit to events with a future DTSTART
? So that past events don't trigger interactions (might be complicated with recurrences).
I'll leave that for another PR 🙈 |
@JohannesGGE @hamza221 reviews pls |
Signed-off-by: Christoph Wurst <[email protected]>
1bace90
to
34d12a7
Compare
If I understand it correctly, normal invitations (adding an event with attendees) and imports of events where the organizer is me, should appear in recently contacted, right? On master, that's the case, in this PR nothing appears in recently contacted :( |
I'll test that another time! |
Summary
Imported events can have attendees to and we wrongly capture them as interactions. Only if the owner of the event is organizer it is their interaction. Anything else can be ignored.
TODO
Checklist