Closed Bug 752867 Opened 9 years ago Closed 9 years ago

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


(Thunderbird :: General, defect)

Not set


(thunderbird13-, thunderbird14+ fixed, thunderbird15 fixed)

Thunderbird 16.0
Tracking Status
thunderbird13 - ---
thunderbird14 + fixed
thunderbird15 --- fixed


(Reporter: standard8, Assigned: standard8)




(1 file)

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.
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
Attached patch The fixSplinter Review
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 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:
Closed: 9 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+
You need to log in before you can comment on or make changes to this bug.