Weather Channel doesn't display forecasts in Spanish

[Víctor R. Ruiz]

Weather Channel doesn't display forecasts in Spanish, redirects to main page.

Test case.
- Switch Ubuntu Touch to Spanish.
- Open Weather app.
- Add a location.
- In main screen, click in forecasts for next 10 days.

Expected result.
- Browser app opens Weather Channel page in the selected location to check the forecasts.

Actual result.
- Browser redirects to Weather Channel's mobile site and asks for location.

[Martin Borho (martin-borho)]

Problem lies on TWC side. The mobile redirect is broken, not only for spanish forecasts.

The forecasts urls to open were implemented like supposed by TWC.

[Víctor R. Ruiz (vrruiz)]

The logic to compose localized forecasts URL's are in components/TabFooter.qml. These are tailored for the desktop and not for Touch.

   function openTWCLink(type) {
        var serviceId = locationTab.locationData.location.services["weatherchannel"],
            loc_name = Qt.locale().name, url = "http://";
        print("Open link for "+type+" - "+loc_name)
        if(loc_name.search(/^de_/) > -1) {
            url += "de.weather.com/";
            if(type) url += ((type === "10day") ? "10tage/" : "stündlich/");

        } else if(loc_name === "en_US" || loc_name === "en-CA") {
            url += "www.weather.com/";
            if(type) url += "weather/"+((type === "10day") ? "tenday/" : "hourbyhour/graph/");

www.weather.com is m.weather.com and espanol.weather.com (as listed in the source code) is m.es.weather.com in the mobile version.

[Martin Borho (martin-borho)]

That's right, but mobile switch was intended to be on TWC side.

[David Planella (dpm)]

We can contact TWC if something is broken on their side. Martin, could you tell me in more detail what they might need to fix?

[Víctor R. Ruiz (vrruiz)]

David: They need to fix the redirection rule. The application composes a localized URL to display the selected country and forecast span. When TWC's servers detect a mobile browser, they redirect to the main site of the country, and forgets about the forecast span. Of course, this can be fixed at the client side, just adding a different set of rules to compose the URL for mobile.

[Martin Borho (martin-borho)]

As Victor says: Either TWC is fixing their mobile redirect or we make the distinction in the app. But as lengthy dealing with TWC is, we should perhaps go with the latter one. Afaik the TWC scope links directly to the mobile page.

[David Planella (dpm)]

Thanks Victor and Martin for the clarification. In that case, Martin, could we do the distinction in the app, so that for RTM we've got a working "redirect", and in parallel I'll start a conversation with TWC? Do you think we could prioritize this bug to be fixed this week?

[Martin Borho (martin-borho)]

If the SDK provides an indicator for being on mobile, for sure. (I don't know if this is available now. )

But since phone is for now the main focus, we can even hardcode the links it to mobile twc, if nothing helps.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

Fix committed into lp:ubuntu-weather-app at revision None, scheduled for release in ubuntu-weather-app, milestone alpha-1