Closed Bug 431126 Opened 16 years ago Closed 15 years ago

Revise calIItipTransport::defaultIdentity


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

Not set


(Not tracked)



(Reporter: Fallen, Assigned: mschroeder)




(1 file)

(Followup from bug 428392 comment #27)

>     /**
>      * Default identity for me within this transport. For example, this is
>      * your default email address in Thunderbird.
>+     *
>+     * XXX This should probably go away, using the default identity is far
>+     * from practical. Only user right now is calUtils's sendItipInvitation
>      */
>     readonly attribute AUTF8String defaultIdentity;
Yeah, when sending an iTIP item we should probably ask the user what identity
to send the email from.  Note that this only has to be done when we generate a
REQUEST or PUBLISH ourselves (i.e. we are the organizer).  This is used as a
workaround for the fact that you can't set the organizer email address in the
event dialog.  However, there are some smart defaults or configuration settings
we could make so that the user could set up their preferred identity to use
when sending meeting requests.  Also, as long as our default behavior is to
show a compose dialog before sending, the user will have a chance to change the
Send From identity via that window.
Instead of asking the organizer each time he creates a new meeting request, you could let us specify the identity in each calendar's properties (and use Thunderbird's default identity unless the user changes it for certain calendars).
(In reply to comment #1)
Bug 431522 implements that.
Depends on: 431522
I think we could now suspend calIItipItem::defaultIdentity.
oops, calIItipTransport::defaultIdentity of course
Assignee: nobody → mschroeder
Attached patch Patch v1Splinter Review
Removed defaultIdentity from the idl, the implementation from the CalDAV provider, and the getter from calItipEmailTransport.js.
Attachment #385655 - Flags: review?(philipp)
Attachment #385655 - Attachment is patch: true
Attachment #385655 - Attachment mime type: application/octet-stream → text/plain
Attachment #385655 - Flags: review?(philipp) → review+
Comment on attachment 385655 [details] [diff] [review]
Patch v1

Pushed to comm-central <>

Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".
Target Milestone: 1.0 → 1.0b1
See Also: → 1562896
You need to log in before you can comment on or make changes to this bug.