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

Proposed by Martin Borho
Status: Merged
Approved by: Raúl Yeguas
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 Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
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

removed mode-param from API request url for daily forecasts

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Raúl Yeguas (neokore) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'components/WeatherApi.js'
--- components/WeatherApi.js 2013-04-27 14:35:21 +0000
+++ components/WeatherApi.js 2013-05-01 14:47:24 +0000
@@ -135,8 +135,8 @@
135 getDailyForecastRequest: function(params) {135 getDailyForecastRequest: function(params) {
136 var request = {136 var request = {
137 type: "daily",137 type: "daily",
138 url: _baseUrl + "forecast/daily?"+params.location.service_id138 url: _baseUrl + "forecast/daily?id="+params.location.service_id
139 +"&mode=daily_compact&cnt=10&units="+params.units,139 +"&cnt=10&units="+params.units,
140 formatter: _buildDailyForecast140 formatter: _buildDailyForecast
141 }141 }
142 return request;142 return request;

Subscribers

People subscribed via source and target branches