Merge lp://qastaging/~artmello/indicator-datetime/indicator-datetime-remove_notification_list_check into lp://qastaging/indicator-datetime
Proposed by
Arthur Mello
| Status: | Needs review |
|---|---|
| Proposed branch: | lp://qastaging/~artmello/indicator-datetime/indicator-datetime-remove_notification_list_check |
| Merge into: | lp://qastaging/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://qastaging/~artmello/indicator-datetime/indicator-datetime-remove_notification_list_check |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Indicator Applet Developers | Pending | ||
|
Review via email:
|
|||
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
