Bug 386067 - Use seperate notifications for each message
Summary: Use seperate notifications for each message
Status: RESOLVED FIXED
Alias: None
Product: kwrited
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.11.1
Platform: Neon Linux
: NOR minor
Target Milestone: ---
Assignee: Plasma Development Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-22 13:20 UTC by Sefa Eyeoglu
Modified: 2018-12-21 14:03 UTC (History)
0 users

See Also:
Latest Commit: https://commits.kde.org/kwrited/c36bf41537e6f7891a065ba10d0ca4829dedd7bc
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Sefa Eyeoglu 2017-10-22 13:20:29 UTC 
If there are multple "unread" messages (for ex. sent with wall), kwrited should create a notification for each message.
This suits the new notification system in 5.11

Kind regards,

Sefa Eyeoglu
Comment 1 Christoph Feck 2018-12-21 02:15:53 UTC 
Are messages separated by some fixed text? Otherwise I doubt it is fixable.
Comment 2 Sefa Eyeoglu 2018-12-21 11:31:57 UTC 
If I run "wall Test" multiple times the messages are combined in one single notification, without any seperator.
Comment 3 David Edmundson 2018-12-21 12:56:23 UTC 
Git commit c36bf41537e6f7891a065ba10d0ca4829dedd7bc by David Edmundson.
Committed on 21/12/2018 at 12:33.
Pushed by davidedmundson into branch 'master'.

Don't group wall notifications

Summary: BUG: 386067

Reviewers: #plasma

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D17721

M  +1    -0    kwrited.cpp

https://commits.kde.org/kwrited/c36bf41537e6f7891a065ba10d0ca4829dedd7bc
Comment 4 Sefa Eyeoglu 2018-12-21 14:03:21 UTC 
Nice! If really need to look into KF5. If I would have known, that it is just one flag I would have created a patch months ago.
Thank you anyway David.