Closed Bug 918493 Opened 7 years ago Closed 7 years ago
Only show accept/decline for unprocessed REQUEST invitations
STR: A little complicated to describe. Requires a server that updates the calendar before the email invitation comes in. See also http://ci-obm.linagora.com/jira/browse/OBMFULL-5269 Steps: 1. boss invites you to a single meeting 2. boss updates the meeting date multiple times 3. calendar server updates the item in your calendar 4. you receive emails for the updates Results: each email shows Accept/Decline buttons Expected: only the latest email, i.e the one that is unprocessed should show the accept/decline buttons.
Ok, this patch takes care. Here is the new behavior. Lets say you have three invitation emails about an event you already have in your calendar (i.e updates): Scenario 1: Server does not process invitations, you get everything via email: - Initially, all three emails say its an update and allow accept/decline/... - Lets say you accept the second one, but then turn it back to NEEDS-ACTION - The first email will say ...event that has already been processed - The second one says ..event that you have not yet responded to - The third one is again an update and allows accept/decline Scenario 2: Server processes invitations - As the server has processed the invitations already, the first and second email will say ...event that has already been processed - The third email matches dtstamp and will allow accept/decline
Attachment #813163 - Flags: review?(matthew.mecca)
Comment on attachment 813163 [details] [diff] [review] Fix - v1 Review of attachment 813163 [details] [diff] [review]: ----------------------------------------------------------------- Untested with server-side processing but looks good codewise. r=mmecca ::: calendar/base/modules/calItipUtils.jsm @@ +1207,5 @@ > let foundAttendee = firstFoundItem.getAttendeeById(att.id); > > + // If the the user hasn't responded to the invitation yet and we > + // are viewing the current representation of the item, show the > + // accep/t/decline buttons. This means newer events will show the There's an extra "/" in the comment.
Attachment #813163 - Flags: review?(matthew.mecca) → review+
Please land attachment 814000 [details] [diff] [review] on c-c, c-a, c-b and comm-esr24.
https://hg.mozilla.org/comm-central/rev/e9910efb86e7 https://hg.mozilla.org/releases/comm-aurora/rev/480f026f2717 https://hg.mozilla.org/releases/comm-beta/rev/103749fcdfca https://hg.mozilla.org/releases/comm-esr24/rev/ebfa64e6c675
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.