Merge lp://staging/~artmello/indicator-datetime/indicator-datetime-remove_notification_list_check into lp://staging/indicator-datetime

Proposed by Arthur Mello
Status: Needs review
Proposed branch: lp://staging/~artmello/indicator-datetime/indicator-datetime-remove_notification_list_check
Merge into: lp://staging/indicator-datetime
Diff against target: 214 lines (+1/-44)
10 files modified
include/datetime/settings-live.h (+0/-1)
include/datetime/settings-shared.h (+0/-1)
include/datetime/settings.h (+0/-1)
include/notifications/notifications.h (+0/-3)
src/notifications.cpp (+0/-11)
src/settings-live.cpp (+0/-12)
src/snap.cpp (+1/-10)
tests/test-notification.cpp (+0/-2)
tests/test-settings.cpp (+0/-1)
tests/test-sound.cpp (+0/-2)
To merge this branch: bzr merge lp://staging/~artmello/indicator-datetime/indicator-datetime-remove_notification_list_check
Reviewer Review Type Date Requested Status
Indicator Applet Developers Pending
Review via email: mp+301216@code.staging.launchpad.net

Commit message

Do not check if calendar can use Notification list
Always display calendar notification to Notification List if notifications are enabled for calendar-app

Description of the change

Do not check if calendar can use Notification list
Always display calendar notification to Notification List if notifications are enabled for calendar-app

To post a comment you must log in.

Unmerged revisions

460. By Arthur Mello

Do not check if calendar can use Notification list
Always display calendar notification to Notification List if notifications are enabled for calendar-app

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