Merge lp:~mr-fiorotto/wingpanel-indicator-notifications/trying-to-fix-1487291 into lp:~wingpanel-devs/wingpanel-indicator-notifications/wingpanel-indicator-notifications
Proposed by
Giuliano Fiorotto
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Adam Bieńkowski | ||||
Approved revision: | 136 | ||||
Merged at revision: | 147 | ||||
Proposed branch: | lp:~mr-fiorotto/wingpanel-indicator-notifications/trying-to-fix-1487291 | ||||
Merge into: | lp:~wingpanel-devs/wingpanel-indicator-notifications/wingpanel-indicator-notifications | ||||
Diff against target: |
13 lines (+2/-1) 1 file modified
src/Services/Notification.vala (+2/-1) |
||||
To merge this branch: | bzr merge lp:~mr-fiorotto/wingpanel-indicator-notifications/trying-to-fix-1487291 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Adam Bieńkowski (community) | code / testing | Approve | |
Review via email: mp+315180@code.launchpad.net |
Commit message
Don't save "transient" notifications
Description of the change
Check if the transient parameter is in the hints Variant, if true don't add the notification to the list.
To post a comment you must log in.
You should be using that check in get_valid method in Notification class instead of using it in add_entry.