Merge lp:~rpadovani/ubuntu-weather-app/1198544 into lp:ubuntu-weather-app/obsolete.trunk

Proposed by Riccardo Padovani
Status: Merged
Approved by: Martin Borho
Approved revision: no longer in the source branch.
Merged at revision: 59
Proposed branch: lp:~rpadovani/ubuntu-weather-app/1198544
Merge into: lp:ubuntu-weather-app/obsolete.trunk
Diff against target: 12 lines (+1/-1)
1 file modified
components/LocationTab.qml (+1/-1)
To merge this branch: bzr merge lp:~rpadovani/ubuntu-weather-app/1198544
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Martin Borho Approve
Review via email:

Commit message

Fixed #1198544

Description of the change

Fixed #1198544

To post a comment you must log in.
Martin Borho (martin-borho) wrote :

Thanks a lot, Bug done!

review: Approve
review: Approve (continuous-integration)
59. By Riccardo Padovani on 2013-07-10

Fixed #1198544. Fixes:

Approved by Ubuntu Phone Apps Jenkins Bot, Martin Borho.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'components/LocationTab.qml'
2--- components/LocationTab.qml 2013-07-03 19:39:29 +0000
3+++ components/LocationTab.qml 2013-07-10 13:55:40 +0000
4@@ -41,7 +41,7 @@
5 // set daily forecasts
6 var dailyForecasts = locationData.daily.results,
7 dailyLength = dailyForecasts.length;
8- for(var x=0;x<dailyLength;x++) {
9+ for(var x=1;x<dailyLength;x++) {
10 dayForecastModel.append({
11 dateRel: "",//Tomorrow",
12 date: formatTimestamp(dailyForecasts[x].timestamp, 'dddd, dd MMMM yyyy'),


People subscribed via source and target branches