Ubuntu Clock App

Merge lp://qastaging/~gang65/ubuntu-clock-app/ubuntu-clock-app-warning-fix into lp://qastaging/ubuntu-clock-app

  1. ubuntu-clock-app-warning-fix
  2. Merge into trunk
Proposed by Bartosz Kosiorek
Status: Merged
Approved by: Nekhelesh Ramananthan
Approved revision: 322
Merged at revision: 321
Proposed branch: lp://qastaging/~gang65/ubuntu-clock-app/ubuntu-clock-app-warning-fix
Merge into: lp://qastaging/ubuntu-clock-app
Diff against target: 120 lines (+17/-17)
2 files modified
app/clock/ClockPage.qml (+1/-1)
po/com.ubuntu.clock.pot (+16/-16)
To merge this branch: bzr merge lp://qastaging/~gang65/ubuntu-clock-app/ubuntu-clock-app-warning-fix
Reviewer Review Type Date Requested Status
Nekhelesh Ramananthan Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+267332@code.qastaging.launchpad.net

Commit message

Silent warning which is displayed during clock run.

Description of the change

Silent warning which is displayed during clock run.

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

PASSED: Continuous integration, rev:322
http://91.189.93.70:8080/job/ubuntu-clock-app-ci/722/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-clock-app-vivid-amd64-ci/67

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-clock-app-ci/722/rebuild

review: Approve (continuous-integration)
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Nice catch! Fixes the warning messages which were silently being ignored. LGTM!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches