Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Extend the VALARM property to include ATTENDEE and SUMMARY properties when ACTION:EMAIL #4671

Closed
miaulalala opened this issue Oct 21, 2022 · 0 comments · Fixed by #4683
Closed

Extend the VALARM property to include ATTENDEE and SUMMARY properties when ACTION:EMAIL #4671

miaulalala opened this issue Oct 21, 2022 · 0 comments · Fixed by #4683
Assignees
@st3iny
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement New feature request feature: caldav nextcloud-gmbh Tickets with importance for Nextcloud Gmbh skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills

Comments

@miaulalala
Copy link
Contributor

miaulalala commented Oct 21, 2022 •
edited
Loading

Is your feature request related to a problem? Please describe.

Currently, we send duplicate reminder emails to all ATTENDEEs of a VEVENT. To improve this, we should implement VALARMs with ATTENDEEs as descibed in the RFC.

Describe the solution you'd like

The new VALARM property for email reminders should look like this

BEGIN:VALARM
ACTION:EMAIL
DESCRIPTION:This is an event reminder
SUMMARY:Alarm notification
ATTENDEE:mailto:[email protected]
TRIGGER:-P0DT2H0M0S
END:VALARM

The item MUST have a summary, which can be the VEVENT summary.

Extend the current calendar object creation to allow setting the EMAIL notification for ATTENDEEs. This can or cannot include the ORGANIZER. We currently don't support defering the attendance of the ORGANIZER to somebody else, so they should be added to the VALARM.

Describe alternatives you've considered

--

Additional context

nextcloud/server#21370
@miaulalala miaulalala added 1. to develop Accepted and waiting to be taken care of enhancement New feature request skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills feature: caldav labels Oct 21, 2022
@miaulalala miaulalala mentioned this issue Oct 21, 2022
Closed
@ChristophWurst ChristophWurst moved this to 🧭 Planning evaluation (dont pick) in 💌 📅 👥 Groupware team Oct 21, 2022
@miaulalala miaulalala added the nextcloud-gmbh Tickets with importance for Nextcloud Gmbh label Oct 24, 2022
@st3iny st3iny self-assigned this Oct 25, 2022
@st3iny st3iny added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Oct 25, 2022
@st3iny st3iny moved this from 🧭 Planning evaluation (dont pick) to 🏗️ In progress in 💌 📅 👥 Groupware team Oct 25, 2022
st3iny added a commit that referenced this issue Oct 25, 2022
@st3iny
Improve handling of email alarms
e2589ed 
Fix #4671

Signed-off-by: Richard Steinmetz <[email protected]>
@st3iny st3iny mentioned this issue Oct 25, 2022
Merged
st3iny added a commit that referenced this issue Oct 25, 2022
@st3iny
Improve handling of email alarms
a2cff2f 
Fix #4671

Signed-off-by: Richard Steinmetz <[email protected]>
Repository owner moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Dec 15, 2022
@st3iny st3iny added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@st3iny st3iny

Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement New feature request feature: caldav nextcloud-gmbh Tickets with importance for Nextcloud Gmbh skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills
Projects
💌 📅 👥 Groupware team
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve handling of email alarms nextcloud/calendar
2 participants
@st3iny @miaulalala