Closed Bug 1708644 Opened 4 years ago Closed 4 years ago

Add API to allow listening to folder events (create/delete/change)

Categories

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

enhancement

Tracking

(thunderbird91 fixed)

RESOLVED FIXED
92 Branch
Tracking Status
thunderbird91 --- fixed

People

(Reporter: buecher, Assigned: john)

Details

Attachments

(1 file)

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

Steps to reproduce:

to create an updated foldertree on the MX side, inform MX side of changes in foldertree

Summary: inform MX of changes in foldertree (folder create or delete) → Add API to allow listening to folder events (create/delete/change)

Note to myself:

  • make sure to include global folder read/unread status in change event

What properties would you like to be notified on changed? This is related to what folder properties should be exposed (those which are not accessible at the moment are marked with missing)

  • name
  • type
  • number of read/unread messages (missing)
  • is favorite (missing)

What else?

Flags: needinfo?(gyunaev)
Flags: needinfo?(buecher)
Assignee: nobody → john
Status: UNCONFIRMED → NEW
Ever confirmed: true

what are the properties we get in the nsifolderlistenner? That has the notify functions, but does not tell for which properties.

just had a lokk at folder properites in the rightclick property menu

-indexed - subscribed - color - synced

what else from that screen and its tabs?

Flags: needinfo?(buecher)
Attachment #9231283 - Attachment description: Bug 1708644 - Add MailFolderInfo type, onCreated, onDeleted, onCopied, onMoved and onFolderInfoChanged events, getFolderInfo, copy and move functions. r=darktrojan → Bug 1708644 - Add MailFolderInfo type, onCreated, onRenamed, onDeleted, onCopied, onMoved and onFolderInfoChanged events, getFolderInfo, copy and move functions. r=darktrojan
Flags: needinfo?(gyunaev)

The patch was already so big, I only added the minimum "favorite", "totalMessageCount" and "unreadMessageCount" information. In a follow-up bug I will try to add color and quota, which seem the most useful remaining information.

Status: NEW → ASSIGNED

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/c9907dd3d108
Add MailFolderInfo type, onCreated, onRenamed, onDeleted, onCopied, onMoved and onFolderInfoChanged events, getFolderInfo, copy and move functions. r=darktrojan

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
Pushed by geoff@darktrojan.net: https://hg.mozilla.org/comm-central/rev/ed18ac99bd02 Stop defining NOTIFICATION_COLLAPSE_TIME in two places. rs=bustage-fix

Comment on attachment 9231283 [details]
Bug 1708644 - Add MailFolderInfo type, onCreated, onRenamed, onDeleted, onCopied, onMoved and onFolderInfoChanged events, getFolderInfo, copy and move functions. r=darktrojan

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
This is a huge addition to the folders API, which took longer than expected and which should be available to 91 ESR

Testing completed (on c-c, etc.):
Risk to taking this patch (and alternatives if risky):
Low.

Please also uplift Geoffs bustage-fix.

Attachment #9231283 - Flags: approval-comm-beta?

Comment on attachment 9231283 [details]
Bug 1708644 - Add MailFolderInfo type, onCreated, onRenamed, onDeleted, onCopied, onMoved and onFolderInfoChanged events, getFolderInfo, copy and move functions. r=darktrojan

[Triage Comment]
Approved for beta

Note "Please also uplift Geoffs bustage-fix." whatever that means.

Attachment #9231283 - Flags: approval-comm-beta? → approval-comm-beta+

Note "Please also uplift Geoffs bustage-fix." whatever that means.

I meant the patch which landed in Comment 8, sorry for being unclear.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: