1654006 - Add an API(s) to get notified when a mail was sent (or sending it failed)

Status: RESOLVED FIXED

(Thunderbird :: Add-Ons: Extensions API, enhancement)

Description

[Int]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/83.0.4103.119 Safari/537.36

Steps to reproduce:

I want to get notified when a mail from the compose window was sent successfully
or if it failed, like it was possible with the legacy API "nsIMsgSendListener"

The API should provide the MessageId of the sent mail enabling to fetch the sent Mail from the Sent-Folder and do further things with it.

Comment 1

[Int]

I made an experiment API which implements the event "onAfterSend" for the compose window
It notifies when a mail was sent and returns the messageId.

https://github.com/int-red/experiment-api-composeevent

Comment 2

[John Bieling (:TbSync)]

Attachment: Bug 1654006 - Add onAfterSend event to compose API. r=mkmelin

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a0226743f89a
Add onAfterSend event to compose API. r=mkmelin

Comment 4

[Geoff Lankow (:darktrojan)]

Unfortunately browser_ext_compose_saveDraft.js and browser_ext_compose_saveTemplate.js are failing on macOS debug after this patch landed.

Comment 5

[John Bieling (:TbSync)]

Found the issue and fix is in the making.

Comment 6

[John Bieling (:TbSync)]

Attachment: Bug 1654006 - Only return headerMessageId for sendNow, to comply with the schema. r=darktrojan

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/d6cf0a914adf
Only return headerMessageId for sendNow, to comply with the schema. r=darktrojan

Comment 8

[John Bieling (:TbSync)]

Comment on attachment 9285737 [details]
Bug 1654006 - Add onAfterSend event to compose API. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
Testing completed (on c-c, etc.):
On Beta since 21 days.

Risk to taking this patch (and alternatives if risky):
The patch is not small and affects sending messages (core) but it has not caused any (known) reports on beta. I would like to make the feature available in 102 already and not wait till 115.

Comment 9

[Wayne Mery (:wsmwk)]

Comment on attachment 9285737 [details]
Bug 1654006 - Add onAfterSend event to compose API. r=mkmelin

[Triage Comment]
Approved for esr102

Comment 10

[Rob Lemley [:rjl]]

Thunderbird 102.3.0:
https://hg.mozilla.org/releases/comm-esr102/rev/2f576d568836
https://hg.mozilla.org/releases/comm-esr102/rev/9bbec4ff8076