Kozea
Add email notification hook
One thing I've been missing from this otherwise great app is server-side email notifications for attendees. Thankfully, it seems we can tap into the hooks functionality to implement email notifications for when events are added/updated and deleted.
When the email hook is enabled, users configure the SMTP server and (optionally) the email body templates for added/updated and deleted event email notifications. When an event is added/updated or deleted, details about the event and the attendees indicated in the VCALENDAR entry will be parsed and used to generate emails, with the ICS file attached to the email.
I've tested this both as a hook plugin (turns out you can extend the hooks even though it's not documented), and as a first-party hook. To collect the VCALENDAR content for sending delete notifications, I had to ultimately introduce a new DeleteHookNotificationItem, so the plugin isn't going to work without modifications to the source code.
The documentation has also been updated.
Please let me know if I missed anything or need to provide any additional information.
Thank you, but can you please extend having TLS + username+password optional as it can be that one simply uses a local running mail system and therefore not requiring TLS and/or username+password.
Also in case of TLS is enabled, similar like ldap_ssl_verify_mode would be helpful having supported.
Also something similiar like imap_security would be helpful.
Please fix also all issues reported by "lint" https://github.com/Kozea/Radicale/actions/runs/15753262702/job/44456578017?pr=1808
Best by running "tox" locally before updating the PR.
https://github.com/Kozea/Radicale/wiki/Development-Testing-Release
also a functional test is broken:
https://github.com/Kozea/Radicale/actions/runs/15753262702/job/44456578046?pr=1808
"lint" is still not happy: https://github.com/Kozea/Radicale/actions/runs/15868136613/job/44739332606?pr=1808
@nwithan8 - lint is still failing, I strongly recommend that you run "tox" before commiting https://github.com/Kozea/Radicale/actions/runs/15893794259/job/44821426622?pr=1808
@nwithan8 - lint is still failing, I strongly recommend that you run "tox" before commiting https://github.com/Kozea/Radicale/actions/runs/15893794259/job/44821426622?pr=1808
Yes, I'm aware. I've put this in draft mode while I work on the feedback in parts. I'll make sure all CI passes before re-opening for review.
Looks all good, but can you please add to DOCUMENTATION.md on each new option and on the hook option
_(>= 3.5.5)_ (so far the next planned version, like shown for other versions)
Looks all good, but can you please add to DOCUMENTATION.md on each new option and on the hook option
_(>= 3.5.5)_(so far the next planned version, like shown for other versions)
Will do, thanks!