D7928: Do not mark messages as unread immediately, makes unread/new filter unusable

Luigi Toscano noreply at phabricator.kde.org
Fri Sep 29 17:14:58 BST 2017


ltoscano reopened this revision.
ltoscano added a comment.
This revision is now accepted and ready to land.


  Two comments (sorry, I'm late):
  
  - If the patch was mostly the one from comment https://bugs.kde.org/show_bug.cgi?id=350731#c39, it should be committed with the wrong author (the person who sent it); if it's the case, please revert and commit again after fixing the author (git commit --author).
  - it should probably go to Applications/17.08
  - in general, please try to use arcanist; there is a disconnection between what was written here (and was part of the evaluation) and what ended in the commit message.

REPOSITORY
  R201 Akregator

REVISION DETAIL
  https://phabricator.kde.org/D7928

To: marten, #kde_pim, dvratil
Cc: ltoscano, anthonyfieroni, cfeck, dvasin, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20170929/7b5f1a40/attachment.html>


More information about the kde-pim mailing list