<table><tr><td style="">albertvaka accepted this revision.<br />albertvaka 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/D13447">View Revision</a></tr></table><br /><div><div><p>The command IDs are quite ugly because they contain "{" which are not safe on URLs. This patch is fine, though, we can change it by changing the generation of those IDs on the desktop side, so they don't have these braces (only for newly-generated IDs).</p>

<p>Eg of a runcommand URI:<br />
 <tt style="background: #ebebeb; font-size: 13px;">kdeconnect://runcommand/_b5e53f8d_0fc6_47f4_9fcc_3bae11da8a0b_/{c3c1cbbe-e182-4ae6-a451-21cf3ff39d79}</tt></p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D13447#inline-75895">View Inline</a><span style="color: #4b4d51; font-weight: bold;">RunCommandUrlActivity.java:68</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">void</span> <span style="color: #004012">error</span><span style="color: #aa2211">(</span><span style="color: #aa4000">int</span> <span class="n">message</span><span style="color: #aa2211">)</span> <span style="color: #aa2211">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">TextView</span> <span class="n">view</span> <span style="color: #aa2211">=</span> <span style="color: #aa4000">new</span> <span class="n">TextView</span><span style="color: #aa2211">(</span><span style="color: #aa4000">this</span><span style="color: #aa2211">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">view</span><span style="color: #aa2211">.</span><span style="color: #354bb3">setText</span><span style="color: #aa2211">(</span><span class="n">message</span><span style="color: #aa2211">);</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Maybe a toast is enough?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>BRANCH</strong><div><div>nfc</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13447">https://phabricator.kde.org/D13447</a></div></div><br /><div><strong>To: </strong>nicolasfella, KDE Connect, albertvaka<br /><strong>Cc: </strong>albertvaka, apol, kdeconnect, KDE Connect, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara, ach<br /></div>