The link to the weather channel is incorrect
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Ubuntu Weather App |
High
|
Martin Borho | ||
Bug Description
click the weather channel in the lower right of the app screen.
it is linking to m.m.weather.com so it does not load.
removing the first m. results in success
Related branches
- Victor Thompson: Approve on 2015-01-23
- Alan Pope 🍺🐧🐱 🦄: Approve on 2015-01-21
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration) on 2015-01-18
-
Diff: 43 lines (+5/-4)2 files modifiedcomponents/LocationTab.qml (+1/-1)
components/TabFooter.qml (+4/-3)
| Dario Cavedon (dcavedon) wrote : | #1 |
| Dario Cavedon (dcavedon) wrote : | #2 |
The link to Open Weather Map is incorrect too (see attachment), it seems the same problem.
| Changed in ubuntu-weather-app: | |
| assignee: | nobody → Martin Borho (martin-borho) |
| status: | New → In Progress |
Fix committed into lp:ubuntu-weather-app at revision None, scheduled for release in ubuntu-weather-app, milestone alpha-1
| Changed in ubuntu-weather-app: | |
| status: | In Progress → Fix Committed |
| dazza5000 (darran-kelinske) wrote : | #4 |
I think this will still be broken for de.weather.com and any of the other locale specific links:
following the logic I don't think that m.de.weather.com will resolve
url = "http://
print("Open link for "+type+" - "+loc_name)
url += "de.weather.com/";
} else if(loc_name === "en_US" || loc_name === "en-CA") {
//url += "www.weather.com/"; //desktop
url += "weather.com/";
} else if(loc_
url += "uk.weather.com/";
} else if(loc_
url += "fr.weather.com/";
} else if(loc_
//url += "espanol.
// if(type) url += "weather/"+((type === "10day") ? "10day-" : "hourByHour-"); //desktop
url += "es.weather.com/";
} else if(loc_name === "ja_JP") {
url += "ja.weather.com/";
} else if(loc_
url += "zh.weather.com/";
} else if(loc_name === "pt_BR") {
url += "br.weather.com/";
} else if(loc_
url += "pt.weather.com/";
} else if(loc_name === "ru_RU") {
url += "ru.weather.com/";
} else {
//url += "www.weather.com/"; //desktop
url += "weather.com/";
}
if(type) url += serviceId;
url += "?par=ubuntu"
print(url)
}
if you go to http://
Herzlich willkommen bei Weather Channel
Damit wir Ihnen die aktuellsten Wetterinformationen bieten können, geben Sie bitte oben im Feld Ihre Postleitzahl oder Ihren Wohnort ein, und wir kümmern uns um alles. Viel Spaß!
| Martin Borho (martin-borho) wrote : | #5 |
Using the links in the app works as expected and you will get send to the specific forecasts of the location, http://
I do not think, you'll ever get only "http://
if(type) url += serviceId;
url += "?par=ubuntu"
print(url)
| Changed in ubuntu-weather-app: | |
| status: | Fix Committed → Fix Released |


Just tried, and I had the same bug (see the attachment).
I'm using mako with Ubuntu 14.10 (r14).