GNOME Bugzilla – Bug 225712
Option to inherit reminder from original scheduled meeting
Last modified: 2009-01-19 13:57:13 UTC
Description of Wish: Setting up a meeting, setting a reminder, and then sending the meeting request from inside the Calendar strips the reminder settings. An option to either inherit or reject (reject by default) the reminder on the email that notifies the invitee would be very useful. I understand that stripping the reminder information is intended, so this bug is a wishlist item.
We might want to only strip reminders that run external applications, not the ones that play sounds and display messages. Those are harmless
still missing in 2.7.3, right. removing old target milestone.
I did this together with bug #220846.
Created attachment 92400 [details] [review] proposed patch for evolution's plugin; I did a bit more here, there are 3 new translation texts, one for "Show time as free", one for "Inherit reminder" and the last one for "Preserve my reminder", that for a case when user has already this event in his/her calendar and has there some alarms, then with this he/she could keep his/her alarms. There could be max two checkboxes, one for "Show time as free" and one for those others, always only one, depending on existence in user's calendar. User could never inherit alarms when the event is already in his/her calendar.
This patch addresses one part of the bug, inheriting/preserving the alarms while accepting the meeting request. The alarms are stripped while sending the meeting request which needs to be addressed too. The procedural alarms should still be stripped while accepting/sending the meeting requests (Please have a look at, 3.5 Security Considerations RFC 2445 and 6.1.6 Procedural Alarms RFC 2446), ignore this if you had already looked at them. Andre would be the best person to comment about the strings. I think a better string can be used for "Inherit reminder". Please provide a separate patch for bug 220846.
the user-visible strings look sexy and i would take them home with me for the weekend if they wouldn't be hard-coded. :-*
I'm waiting for review of patch at bug 220846, because those changes are required to do this.
Created attachment 100236 [details] [review] propsoed evo patch for evolution; it's quite longer because I changed some "core" functions. See ChangeLog entries for more details. But it works, at least the parts I was checking.
Bumping version to a stable release.
Please commit the patch. I have gone through the patch but have not tested it. The patch looks good to commit. Please announce the string changes.
Committed to trunk. Committed revision 37097.