Support disabling of what's new prompts via the update server snippets

RESOLVED FIXED in Thunderbird 16.0

Status

Thunderbird
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
Thunderbird 16.0

Thunderbird Tracking Flags

(thunderbird13-, thunderbird14+ fixed, thunderbird15 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Firefox has a mechanism (implemented in bug 538331) so that it is an update-generation time decision as to whether or not to push a what's new page for a particular release.

With the new automation switch-over, Thunderbird will also gain the necessary update server support for this and we already know this is something that we want to have.

The bulk of this appears to be porting the checks in Firefox's nsBrowserContentHandler.js / getPostUpdateOverridePage function to an appropriate place in Thunderbird.

We can then use that to control showing the what's new page in specialTabs.js / openSpecialTabsOnStartup.

We should also redirect the what's new help menu item like we did in bug 746320.
(Assignee)

Comment 1

5 years ago
Looks like we won't need this for TB 13, but we should still look to get in place for subsequent releases.
tracking-thunderbird13: ? → -
(Assignee)

Updated

5 years ago
tracking-thunderbird14: ? → +

Comment 2

5 years ago
Hi, I'm new to Mozilla and I'm looking for something to start on. Could this be my first bug?
(In reply to awjohnston1 from comment #2)
> Hi, I'm new to Mozilla and I'm looking for something to start on. Could this
> be my first bug?

Of course. If you need help either ask here or in #maildev on irc.mozilla.org.
(Assignee)

Comment 4

5 years ago
The only thing to note, is that we now need this by 9th July at the latest so that it can be included in the next release for which we'll need it. If you don't think you can do it again, I can find other bugs for you - just email me direct.
(Assignee)

Updated

5 years ago
Assignee: nobody → mbanner
Whiteboard: [mentor=standard8][lang=js]
(Assignee)

Updated

5 years ago
Keywords: helpwanted
(Assignee)

Comment 5

5 years ago
Created attachment 638483 [details] [diff] [review]
The fix

This updates us to check the app.update.postupdate value to see if we've actually updated and finished updating, and then also check to see if updates.xml has a actions="silent" parameter in it. If it does have the silent parameter, then it doesn't show the update.

To test this, I copied an updates.xml file from a Thunderbird build (mac has it in the application dir), and added 'actions="silent"' to the first <update... element in that file. I then also set the app.update.postupdate as a boolean (value doesn't matter).

I'm looking to get this in tomorrow's beta so that we can verify the whole route from the AUS server to Thunderbird before we push it out on the release channel.
Attachment #638483 - Flags: review?(dbienvenu)

Comment 6

5 years ago
Comment on attachment 638483 [details] [diff] [review]
The fix

there's a dump statement still in the patch, and you could probably use let instead of var for um.
Attachment #638483 - Flags: review?(dbienvenu) → review+
(Assignee)

Comment 7

5 years ago
Checked in with nits addressed:

https://hg.mozilla.org/comm-central/rev/82179c85043d
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 16.0
(Assignee)

Comment 8

5 years ago
Comment on attachment 638483 [details] [diff] [review]
The fix

[Triage Comment]
We want this on beta to test for the next release.
Attachment #638483 - Flags: approval-comm-beta+
Attachment #638483 - Flags: approval-comm-aurora+
(Assignee)

Comment 9

5 years ago
Checked in:

https://hg.mozilla.org/releases/comm-aurora/rev/4e3fc600fcc2
https://hg.mozilla.org/releases/comm-beta/rev/5987adbec4ac
status-thunderbird14: --- → fixed
status-thunderbird15: --- → fixed
You need to log in before you can comment on or make changes to this bug.