Merge lp:~martin-borho/ubuntu-weather-app/API-url-fix into lp:ubuntu-weather-app/obsolete.trunk

Proposed by Martin Borho on 2013-05-01
Status: Merged
Approved by: Raúl Yeguas on 2013-05-01
Approved revision: 16
Merged at revision: 15
Proposed branch: lp:~martin-borho/ubuntu-weather-app/API-url-fix
Merge into: lp:ubuntu-weather-app/obsolete.trunk
Diff against target: 14 lines (+2/-2)
1 file modified
components/WeatherApi.js (+2/-2)
To merge this branch: bzr merge lp:~martin-borho/ubuntu-weather-app/API-url-fix
Reviewer Review Type Date Requested Status
Raúl Yeguas 2013-05-01 Approve on 2013-05-01
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve on 2013-05-01
Review via email: mp+161830@code.launchpad.net

Commit message

Fixed in API request url for daily forecast

Description of the change

Fix in API request url

To post a comment you must log in.
16. By Martin Borho on 2013-05-01

removed mode-param from API request url for daily forecasts

Raúl Yeguas (neokore) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'components/WeatherApi.js'
2--- components/WeatherApi.js 2013-04-27 14:35:21 +0000
3+++ components/WeatherApi.js 2013-05-01 14:47:24 +0000
4@@ -135,8 +135,8 @@
5 getDailyForecastRequest: function(params) {
6 var request = {
7 type: "daily",
8- url: _baseUrl + "forecast/daily?"+params.location.service_id
9- +"&mode=daily_compact&cnt=10&units="+params.units,
10+ url: _baseUrl + "forecast/daily?id="+params.location.service_id
11+ +"&cnt=10&units="+params.units,
12 formatter: _buildDailyForecast
13 }
14 return request;

Subscribers

People subscribed via source and target branches