Merge lp://qastaging/~nik90/ubuntu-clock-app/fix-alarm-list-refresh into lp://qastaging/ubuntu-clock-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Approved by: Nekhelesh Ramananthan
Approved revision: 362
Merged at revision: 359
Proposed branch: lp://qastaging/~nik90/ubuntu-clock-app/fix-alarm-list-refresh
Merge into: lp://qastaging/ubuntu-clock-app
Diff against target: 51 lines (+5/-5)
3 files modified
app/alarm/AlarmList.qml (+1/-1)
app/alarm/EditAlarmPage.qml (+3/-4)
debian/changelog (+1/-0)
To merge this branch: bzr merge lp://qastaging/~nik90/ubuntu-clock-app/fix-alarm-list-refresh
Reviewer Review Type Date Requested Status
Zsombor Egri (community) Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Ubuntu Clock Developers Pending
Review via email: mp+269047@code.qastaging.launchpad.net

Commit message

Refresh alarmModel when the alarm being edited is saved to ensure that the listview shows updated labels.

Description of the change

Refresh alarmModel when the alarm being edited is saved to ensure that the listview shows updated labels.

To post a comment you must log in.
361. By Nekhelesh Ramananthan

Minor changelog typo fixed

362. By Nekhelesh Ramananthan

Revert making variable readonly

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
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Zsombor Egri (zsombi) wrote :

Looks good, and right way to refresh the list.

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