<table><tr><td style="">piggz 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/D10703" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Guys, whats your thoughts about this?</p>

<p>Ive marked everything as 'done' so far, and quite happy to have removed the hack which compiled the declarative plugin into the app, and installed it properly.  I should have done this from the start!</p>

<p>Also fixed the qAsConst issue by taking the code from qt 5.7. (btw, your min qt version is 5.2, which surely cant work?)</p>

<p>Perhaps the next change could be to abstract out the notifications code, so there was just one set of ifdef's for the different notification libs, but I dont want this to get too big a diff.</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/D10703" rel="noreferrer">https://phabricator.kde.org/D10703</a></div></div><br /><div><strong>To: </strong>piggz, KDE Connect, apol<br /><strong>Cc: </strong>kossebau, mtijink, vonreth, apol, KDE Connect, nicolasfella, vhyvert, adeen-s, SemperPeritus, ahmedbesbes, daniel.z.tg, jeanv, tfella, aboudhar, seebauer, bugzy, MayeulC, menasshock, ach<br /></div>