<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/D17214">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/D17214#367762" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17214#367762</a>, <a href="https://phabricator.kde.org/p/andyholmes/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@andyholmes</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>There is also an error when you type the phone number for a contact with whom there is already a conversation</p></blockquote>

<p>Something that might help when considering <tt style="background: #ebebeb; font-size: 13px;">address</tt> vs <tt style="background: #ebebeb; font-size: 13px;">thread_id</tt>, is to notice that if you receive an SMS on Android from any contact (ie. Mom/Dad) who share a phone number, Google/Android will just pick one and the messages will go there.</p>

<p>In other words, it may no even be possible to have two different <tt style="background: #ebebeb; font-size: 13px;">thread_id</tt>'s for the same <tt style="background: #ebebeb; font-size: 13px;">address</tt>, so the <tt style="background: #ebebeb; font-size: 13px;">thread_id</tt> is really only necessary for fetching messages. In the GSConnect UI, new incoming messages (whether for a new or existing conversation) are "routed" by <tt style="background: #ebebeb; font-size: 13px;">address</tt>, not by <tt style="background: #ebebeb; font-size: 13px;">thread_id</tt>, and I haven't found a situation where that breaks yet.</p></div>
</blockquote>

<p>Oh. Hm. All of our SMS app works on thread_id, with the assumption that they would be unique, or at least the desktop GUI would match the phone GUI. If I'm understanding what you're saying correctly, we're likely going to hit a wall at some point. In any case, fixing that is outside of the scope of this current patch.</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/D17214">https://phabricator.kde.org/D17214</a></div></div><br /><div><strong>To: </strong>turx, albertvaka, nicolasfella, sredman, KDE Connect<br /><strong>Cc: </strong>andyholmes, apol, kdeconnect, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara<br /></div>