<table><tr><td style="">sredman added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15108">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D15108#316416" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15108#316416</a>, <a href="https://phabricator.kde.org/p/nicolasfella/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@nicolasfella</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Is it really necessary? We use a ProxyModel to sort the messages in the view</p></div>
</blockquote>

<p>Yes because in the app we ask the ConversationDBusInterface for the 10 most recent messages. Since the list was badly sorted it would return the 10 most recent messages <strong>at the time they were requested</strong>. If new messages came in, they would not be shown (you made a reference to this earlier as having to disconnect and reconnect to see new messages). They would be correctly sorted in the QML app, but the app was already receiving stale data. This makes the situation a little better in that you only have to change which conversation is being viewed to see new messages.</p>

<p>I think the method of getting messages into the app might need to be redesigned at some point regardless, but I expect we will want a sorted representation in the model no matter what</p>

<p>I have drawn what was happening before: Hopefully that helps show why this is a needed change: <a href="https://phabricator.kde.org/F6220963" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6220963: 2018-08-28 KDE Connect Sorted Message Drawing.pdf</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15108">https://phabricator.kde.org/D15108</a></div></div><br /><div><strong>To: </strong>sredman<br /><strong>Cc: </strong>nicolasfella, kdeconnect, rmenezes, julioc, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara, apol<br /></div>