Closed Bug 1594246 Opened 5 months ago Closed 5 months ago

Remove mail.addr_book.newDirType pref for creating address books of a different type

Categories

(MailNews Core :: Address Book, task)

task
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 72.0

People

(Reporter: darktrojan, Assigned: darktrojan)

References

Details

Attachments

(1 file)

In bug 1572324 I created a pref for choosing whether Mork or JS address books should be created. This pref should have changed to JS by default (probably when bug 1576525 landed), but this never happened, and it is near-useless now anyway as Mork address books are forcibly converted at start-up. It's time for the pref to go.

Blocks: 1588831
Attachment #9106781 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9106781 [details] [diff] [review]
1594246-remove-dirtype-pref-1.diff

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

Looks good, r=mkmelin
Attachment #9106781 - Flags: review?(mkmelin+mozilla) → review+
Blocks: 1572324

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/126714837c0b
Remove mail.addr_book.newDirType pref for creating address books of a different type. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 72.0
Comment on attachment 9106781 [details] [diff] [review]
1594246-remove-dirtype-pref-1.diff

Since we're *still* waiting for beta 4.
Attachment #9106781 - Flags: approval-comm-beta?
Comment on attachment 9106781 [details] [diff] [review]
1594246-remove-dirtype-pref-1.diff

Anyone managing those approvals :-( - I can see the landings are doing just fine.
Attachment #9106781 - Flags: approval-comm-beta? → approval-comm-beta+
Comment on attachment 9106781 [details] [diff] [review]
1594246-remove-dirtype-pref-1.diff

Missed TB 71 beta 4.
Attachment #9106781 - Flags: approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.