1231189 - Latest Firefox for iOS release notes URL redirect includes desktop version number

Status: RESOLVED FIXED

(www.mozilla.org :: Bedrock, defect)

Description

[Steven Garrity [:sgarrity]]

In a bedrock template, I'm trying to output a link to the latest Firefox for iOS release notes with the following:

 url('firefox.notes', platform='ios')

This returns a link to /firefox/ios/notes/ (so far so good).

However, that URL then gets redirected to https://www.mozilla.org/en-US/firefox/ios/42.0.2/releasenotes/

(42.0.2 being the latest desktop Firefox release at the time of this writing)

Somewhere there is a redirect affecting the ios release notes that should be limited to the desktop, and an iOS specific redirect setup with the appropriate release version.

Perhaps related to https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/urls.py#L116

Comment 1

[Josh Mize [:jgmize]]

Last time I checked, there was no latest version of FFiOS in product details, and all links to release notes had to be hard coded.

Comment 2

[Jonathan Petto [:jpetto]]

Still investigating, but it appears the version returned matches the latest Android release. FirefoxIOS.latest_version [1] is using the same mobile_details method as FirefoxAndroid.latest_version [2]. This mobile_details method maps directly to a product-details JSON file (mobile_details.json [3]), which appears to only refer to Android. I don't see any data in product-details related to iOS builds, which may be the heart of the problem.

[1] https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/firefox_details.py#L277
[2] https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/firefox_details.py#L256
[3] http://viewvc.svn.mozilla.org/vc/libs/product-details/json/mobile_details.json?view=markup

Comment 3

[Jonathan Petto [:jpetto]]

If we don't have plans to put iOS information into product-details any time soon, perhaps as a fix we can update FirefoxIOS.latest_version [1] to return a value specified in settings? We should do something to rectify that currently broken method.

[1] https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/firefox_details.py#L275

Comment 4

[Josh Mize [:jgmize]]

(In reply to Jon Petto [:jpetto] from comment #3)
> If we don't have plans to put iOS information into product-details any time
> soon, perhaps as a fix we can update FirefoxIOS.latest_version [1] to return
> a value specified in settings? We should do something to rectify that
> currently broken method.

That seems like a reasonable near term approach to me.

Comment 5

[Jonathan Petto [:jpetto]]

Attachment: GitHub PR

Comment 6

[[github robot]]

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

https://github.com/mozilla/bedrock/commit/c0191b829b99669b1ae5aefc92440fdd2c7c14c9
[fix bug 1231189] Return correct value from FirefoxIOS.latest_version

https://github.com/mozilla/bedrock/commit/b9871f9c0680d7929e371248d0f2f0e7093328b4
Merge pull request #3770 from jpetto/bug-1231189-fix-fxios-releasenotes-url

[fix bug 1231189] Return correct value from FirefoxIOS.latest_version