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

RESOLVED FIXED in Thunderbird 16.0


5 years ago
5 years ago


(Reporter: standard8, Assigned: standard8)


Thunderbird 16.0

Thunderbird Tracking Flags

(thunderbird13-, thunderbird14+ fixed, thunderbird15 fixed)



(1 attachment)

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.
Looks like we won't need this for TB 13, but we should still look to get in place for subsequent releases.
tracking-thunderbird13: ? → -
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
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: nobody → mbanner
Whiteboard: [mentor=standard8][lang=js]
Keywords: helpwanted
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+
Checked in with nits addressed:
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 16.0
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+
Checked in:
status-thunderbird14: --- → fixed
status-thunderbird15: --- → fixed
You need to log in before you can comment on or make changes to this bug.