<table><tr><td style="">nicolasfella requested changes to this revision.<br />nicolasfella added a comment.<br />This revision now requires changes to proceed.
</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/D14248">View Revision</a></tr></table><br /><div><div><p>Hi, thanks for your interest in KDE Connect!</p>

<p>You can find instructions on how to build the KDE Connect desktop side in <a href="https://nicolasfella.wordpress.com/2018/03/30/building-kde-connect/" class="remarkup-link" target="_blank" rel="noreferrer">https://nicolasfella.wordpress.com/2018/03/30/building-kde-connect/</a> . Feel free to ask me about anything unclear.</p>

<p>For the future I recommend using the arc command line tool to upload patches. It makes our and your life easier.</p>

<p>Also, please remove all unrelated changes, i.e. the Log calls, before uploading patches.</p>

<p>Obviously the ping plugin is not the right place to implement the functionality. You can find instructions on how to create a new plugin in the plugins/ folders README file in the desktop repo.</p>

<p>Note that we already have a url handler for e.g. phone urls. It can be invoked by kdeconnect-handler</p>

<p>For discussion around KDE Connect we have a Telegram group that you can join via <a href="https://t.me/joinchat/AOS6gA37orb2dZCLhqbZjg" class="remarkup-link" target="_blank" rel="noreferrer">https://t.me/joinchat/AOS6gA37orb2dZCLhqbZjg</a></p></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/D14248">https://phabricator.kde.org/D14248</a></div></div><br /><div><strong>To: </strong>vishalgupta, nicolasfella<br /><strong>Cc: </strong>nicolasfella, kdeconnect, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara, ach, apol<br /></div>