whatsnew page for ESR24 -> ESR31 updates doesn't show Australis tour

RESOLVED FIXED

Status

www.mozilla.org
Bedrock
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: nthomas, Assigned: agibson)

Tracking

Production
x86
All
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [kb=1630133])

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Users of 24.x.y (ESR24) are shown the whatsnew page (eg)
  https://www.mozilla.org/en-US/firefox/31.4.0/whatsnew/?oldversion=24.8.0
when they update to an ESR31 build. When this was first set up (bug 1018694) this had the Australis tour on it, but now it has 'Committed to you, your privacy and an open Web' and Firefox sync.
:agibson - can you confirm if there's anything we can do on the bedrock side?
Flags: needinfo?(agibson)
(Assignee)

Comment 2

3 years ago
Since the original Australis /whatsnew tour was made we have done several iterations and updates in various revisions of Firefox since. We made the decision not to indefinitely support old versions of the /whatsnew tour, as there have since been many updates to UITour and supporting older versions becomes more difficult over time. Also our logic for showing /whatsnew templates is now getting quite complex [1].

That said, if there are still people upgrading from pre 29 versions to an ESR such as this, we could add in another special case for people updating to 31.x from < 29.

[1] https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/views.py#L412
Flags: needinfo?(agibson)
(Assignee)

Updated

3 years ago
Assignee: nobody → agibson
Status: NEW → ASSIGNED
(Assignee)

Updated

3 years ago
Whiteboard: [kb=1630133]
(Assignee)

Comment 3

3 years ago
Created attachment 8548867 [details] [review]
GitHub pull request
(Reporter)

Comment 4

3 years ago
Thanks for writing the patch to restore the tour. I'd filed this because release engineering hadn't heard that we should stop doing whatsnew for esr24 users. Certainly it would make our work a little easier if we could stop, but it's release management and other's call. And we won't ever update every user, especially on esr where the user may not have that choice, but it may well have been long enough that most people have.
(Assignee)

Comment 5

3 years ago
(In reply to Nick Thomas [:nthomas] from comment #4)
> Thanks for writing the patch to restore the tour. I'd filed this because
> release engineering hadn't heard that we should stop doing whatsnew for
> esr24 users. Certainly it would make our work a little easier if we could
> stop, but it's release management and other's call. And we won't ever update
> every user, especially on esr where the user may not have that choice, but
> it may well have been long enough that most people have.

Thanks, Nick!

Fwiw, I don't think we we're aware whatsnew was being shown for ESR24 users when updating. Apologies if this got missed somehow. We can continue to keep showing it for now. If this needs to change in the future we'll let you know.

Comment 6

3 years ago
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/f3bd8c87d7cbc5cbad8c61e2f1ac05d6c6ea1fed
[fix bug 1121209] whatsnew page for ESR24 -> ESR31 updates doesn't show Australis tour

https://github.com/mozilla/bedrock/commit/eaab8266964615c31e3cdbccb86d77600763712b
Merge pull request #2657 from alexgibson/bug1121209-whatsnew-esr31

[fix bug 1121209] whatsnew page for ESR24 -> ESR31 updates doesn't show Australis tour

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Reporter)

Comment 7

2 years ago
FYI, we're no longer showing a whatsnew page to ESR24 users when they update to 31.
You need to log in before you can comment on or make changes to this bug.