<table><tr><td style="">sredman accepted this revision.<br />sredman added a comment.<br />This revision is now accepted and ready to land.
</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/D17126">View Revision</a></tr></table><br /><div><div><p>Sorry this took so long</p>

<p>I finally had a chance to check, and you are correct. The 'bug' where the notification plugin settings are not per-device is there on the master build too. In that case, it's a problem that should be solved, but not as part of this patch :)</p>

<p>I think this is good work. There is one part that's missing, though. If you notice, when you build from master or you use the Play Store build, then you upgrade to a build with this patch, the app will crash with some error about missing a table. This is because we don't currently have an onUpgrade which handles putting in the new tables, so someone upgrading would have to wipe their app data! This is obviously not great.</p>

<p>There is a solution. If you want to learn about it, look here is the relevant tutorial: <a href="https://thebhwgroup.com/blog/how-android-sqlite-onupgrade" class="remarkup-link" target="_blank" rel="noreferrer">https://thebhwgroup.com/blog/how-android-sqlite-onupgrade</a></p>

<p>I have made a revision which depends on this one: <a href="https://phabricator.kde.org/D17521" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17521</a>. We can merge that revision right after merging this one, then everything should be working! :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D17126</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17126">https://phabricator.kde.org/D17126</a></div></div><br /><div><strong>To: </strong>alexkovrigin, sredman, KDE Connect<br /><strong>Cc: </strong>sredman, alexkovrigin, kdeconnect, mschroeder, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara, apol<br /></div>