Closed
Bug 753324
Opened 13 years ago
Closed 13 years ago
Show Attendees in the invitation template
Categories
(Calendar :: E-mail based Scheduling (iTIP/iMIP), defect)
Calendar
E-mail based Scheduling (iTIP/iMIP)
Tracking
(Not tracked)
RESOLVED
FIXED
1.7
People
(Reporter: Fallen, Assigned: Fallen)
References
Details
(Whiteboard: [wanted-1.2.x])
Attachments
(1 file, 1 obsolete file)
|
21.14 KB,
patch
|
laurent
:
review+
|
Details | Diff | Splinter Review |
We should give the user the chance to see the Attendee (and status) directly in the invitation email. This patch takes care.
A few piggyback fixes:
* Move attendees icon CSS into separate file so it can be re-used independent of the event dialog
* Use the toString() method on attendees more, since it does the mailto: stripping already.
* Give the organizer an icon too.
Something I noticed is that if the organizer is also an Attendee, he is shown twice. I will take care of this in a separate patch.
Note this patch requires bug 753305 and its patch to be applied.
|
Assignee | |
Comment 1•13 years ago
|
||
Attachment #622648 -
Flags: review?(laurent)
|
|
Assignee | |
Comment 2•13 years ago
|
||
Forgot to add calendar-attendees.css and debitrotted
Attachment #622648 -
Attachment is obsolete: true
Attachment #622648 -
Flags: review?(laurent)
Attachment #623057 -
Flags: review?(laurent)
|
||
Comment 3•13 years ago
|
||
Comment on attachment 623057 [details] [diff] [review]
Fix - v2
Review of attachment 623057 [details] [diff] [review]:
-----------------------------------------------------------------
Ok for me.
Attachment #623057 -
Flags: review?(laurent) → review+
|
|
Assignee | |
Comment 4•13 years ago
|
||
Pushed to comm-central changeset b79072737e40
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.7
|
|
Assignee | |
Updated•13 years ago
|
Whiteboard: [wanted-1.2.x]
You need to log in
before you can comment on or make changes to this bug.
Description
•