<table><tr><td style="">daniel.z.tg 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/D7652" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Effects of most recent changes, tested with one computer and one phone:</p>

<p>Unaffected:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Normal WiFi should still work.</li>
</ul>

<p>Internal:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Different broadcasts are received</li>
<li class="remarkup-list-item">BackgroundService is now static (it was already singleton before)</li>
<li class="remarkup-list-item">BackgroundService has been slightly optimized (not benchmarked)</li>
<li class="remarkup-list-item">Link-local broadcast addresses are now used instead of the generic "255.255.255.255"</li>
<li class="remarkup-list-item">Mobile network detection is now incompatible network detection, and is now done by checking if no broadcast addresses can be found</li>
<li class="remarkup-list-item">Translations for incompatible network detection are missing</li>
</ul>

<p>Working better:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">WiFi tethering<ul class="remarkup-list">
<li class="remarkup-list-item">Enabling/disabling tethering is correctly in sync with detection of it as a compatible network</li>
<li class="remarkup-list-item">Peer detection & connection works partially<ul class="remarkup-list">
<li class="remarkup-list-item">Should fully work on Android L and below (untested)</li>
<li class="remarkup-list-item">Fully working on LineageOS 14.1 (Android N) and some other ROMs</li>
<li class="remarkup-list-item">Requires refresh for other devices on Android M and above</li>
</ul></li>
<li class="remarkup-list-item">Peer disconnection requires disabling tethering to be noticed</li>
</ul></li>
<li class="remarkup-list-item">Bluetooth tethering sometimes works fully, and always works partially<ul class="remarkup-list">
<li class="remarkup-list-item">Peer discovery & connection sometimes requires refresh</li>
<li class="remarkup-list-item">Peer disconnection is fully working</li>
<li class="remarkup-list-item">Enabling/disabling tethering is not applicable. Network compatibility is detected upon peer connection and disconnection.</li>
</ul></li>
<li class="remarkup-list-item">USB tethering works partially<ul class="remarkup-list">
<li class="remarkup-list-item">Peer discovery usually requires a refresh</li>
<li class="remarkup-list-item">Peer connection may require refresh</li>
<li class="remarkup-list-item">Peer disconnection is fully working</li>
<li class="remarkup-list-item">Enabling tethering sometimes requires a refresh to network comparability. When peers are detected before refreshing, the GUI still says that no compatible networks are available.</li>
<li class="remarkup-list-item">Disabling tethering is fully working is correctly in sync with detection of a compatible network being removed</li>
</ul></li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7652" rel="noreferrer">https://phabricator.kde.org/D7652</a></div></div><br /><div><strong>To: </strong>daniel.z.tg, KDE Connect<br /><strong>Cc: </strong>albertvaka, daniel.z.tg, jeanv, tfella, aboudhar, seebauer, bugzy, progwolff, MayeulC, menasshock, ach, apol, hkaelberer<br /></div>