Skip to content

Fix Bug 952549 - Update ESR versions in the Firefox details#1549

Merged
jgmize merged 1 commit intomozilla:masterfrom
kyoshino:update-esr-versions
Mar 27, 2014
Merged

Fix Bug 952549 - Update ESR versions in the Firefox details#1549
jgmize merged 1 commit intomozilla:masterfrom
kyoshino:update-esr-versions

Conversation

@kyoshino
Copy link
Copy Markdown
Contributor

The firefox_details.esr_major_versions array is used for version checks. It should be up-to-date.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should only be checking for the latest (24), not any ESR release (10,17,24)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, looks like we have to update this manually along with the range above. Both 10 and 17 are now EOL, and the esr_major_versions mean the latest Firefox ESR versions as per context_processors.py.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The logic behind this is now being discussed over on Bug 952549.

@kyoshino
Copy link
Copy Markdown
Contributor Author

Closing until the final decision is made in Bug 952549.

@kyoshino kyoshino closed this Feb 27, 2014
@kyoshino
Copy link
Copy Markdown
Contributor Author

Firefox 10 and 17 are totally obsolete. Let's do this and go ahead with #1548.

@kyoshino kyoshino reopened this Mar 26, 2014
jgmize added a commit that referenced this pull request Mar 27, 2014
Fix Bug 952549 - Update ESR versions in the Firefox details
@jgmize jgmize merged commit ee5a9a6 into mozilla:master Mar 27, 2014
@kyoshino kyoshino deleted the update-esr-versions branch March 28, 2014 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants