420831 - On sending updated iTIP message, the subject does not indicate 'updated'

Status: RESOLVED FIXED

(Calendar :: E-mail based Scheduling (iTIP/iMIP), enhancement)

Description

[Marcel Berteler]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.12) Gecko/20080201 Firefox/2.0.0.12
Build Identifier: 20080302

On updating an event and sending out invitations, the subject does not indicate this is an updated event. 



Reproducible: Always

Steps to Reproduce:
1.Update an event that has has attendees that are invited via email
2.Save and close
Actual Results:  
The email that will be send out has the following subject:

"Event Invitation: EVENT TITLE"


Expected Results:  
As this is an updated event, the subject should indicate this:

"Updated: EVENT TITLE"

Comment 1

[Pete Riley]

It would be nice if this enhancement request was generalized to also include "Accepted: Event Title", "Declined: Event Title", etc.

Currently (as far as I know) it's only possible to determine if another Lightning user has accepted or declined my invitation by (1) clicking the Update button in the iMIP bar, (2) switching to Calendar Mode, (3) finding the event in the view, (4) opening the event dialog, and (5) clicking on the blue link of the attendees and looking at the icon next to the person.  In other words, too painful.

Comment 2

[PMARTINAK]

Attachment: [after beta][l10n]Patch v1 with some usuability improvement

The proposed patch change message subject for updated event to:
Updated Event Invitation:<title>

This patch also change subject of the response message when accepting, declining or cancelling an invitation.
Accepted : <title>
Declined : <title>
Tentative: <title>

This patch apply to lightning 1.0b3pre/1.9.2/20110131.

Comment 3

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Comment on attachment 509403 [details] [diff] [review]
[after beta][l10n]Patch v1 with some usuability improvement

This patch contains strings, we are in string freeze so I can't get to this until after the coming release. Thank you for the patch though!

Comment 4

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Fix - v2

This patch is an update on the last, but basically does the same. I'd still appreciate a review on my changes.

Comment 5

[Matthew Mecca [:mmecca]]

Comment on attachment 589195 [details] [diff] [review]
Fix - v2

Review of attachment 589195 [details] [diff] [review]:
-----------------------------------------------------------------

r=mmecca with one typo fixed

::: calendar/itip/calItipEmailTransport.js
@@ +159,5 @@
> +                            subjectKey = "itipReplySubjectAccept";
> +                            bodyKey = "itipReplyBodyAccept";
> +                            break;
> +                        case "TENTATIVE":
> +                            subjectKey = "itipReplySubjectTenative";

should be "itipReplySubjectTentative" (missing a 't')

Comment 6

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Pushed to comm-central changeset c557ae46d0dd