Accounts API: mark special folders with their type

RESOLVED FIXED in Thunderbird 66.0

Status

enhancement
RESOLVED FIXED
4 months ago
4 months ago

People

(Reporter: darktrojan, Assigned: darktrojan)

Tracking

(Blocks 1 bug)

unspecified
Thunderbird 66.0
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Assignee

Description

4 months ago

We should have a field on some special types of folders (junk, trash, outbox, etc.) which says what type of folder they are, so they can be identified easily.

Assignee

Updated

4 months ago
Blocks: webext-tb
Depends on: 1488176
Assignee

Comment 1

4 months ago
Attachment #9036840 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9036840 [details] [diff] [review]
1520367-webext-folder-type-1.diff

Review of attachment 9036840 [details] [diff] [review]:
-----------------------------------------------------------------

Stealing the review. One minor comment to consider:

::: mail/components/extensions/schemas/accounts.json
@@ +42,5 @@
> +          "type": {
> +            "type": "string",
> +            "optional": true,
> +            "description": "The type of folder, for several common types.",
> +            "enum": ["inbox", "drafts", "sent", "trash", "templates", "archives", "junk", "outbox"]

If you look at other API schemas, they tend to define a type and then $ref it. See for example https://searchfox.org/comm-central/source/mail/components/extensions/schemas/windows.json#11

It is possible this only happens when it is actually used more than once, I haven't verified this.
Attachment #9036840 - Flags: review?(mkmelin+mozilla) → review+

Comment 3

4 months ago

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/23ef8a434c94
Accounts API: mark special folders with their type; r=Fallen DONTBUILD

Status: ASSIGNED → RESOLVED
Last Resolved: 4 months ago
Resolution: --- → FIXED
Assignee

Updated

4 months ago
Target Milestone: --- → Thunderbird 66.0
Assignee

Comment 4

4 months ago

I discovered one place where this breaks things because the schema asks for an object, not a MailFolder. Will post a patch shortly.

Assignee

Updated

4 months ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee

Comment 5

4 months ago
Attachment #9037928 - Flags: review?(philipp)
Attachment #9037928 - Flags: review?(philipp) → review+
Assignee

Updated

4 months ago
Keywords: checkin-needed

Comment 6

4 months ago

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/21ddeb46254b
followup - Change mailTabs schema to use defined type; r=Fallen

Status: REOPENED → RESOLVED
Last Resolved: 4 months ago4 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.