<table><tr><td style="">albertvaka 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/D8942" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I'm a bit busy, and haven't tested it, but looks good to me. Thanks for the patch! <a href="https://phabricator.kde.org/p/apol/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;" rel="noreferrer">@apol</a> can you merge it?</p>
<p><a href="https://phabricator.kde.org/p/mtijink/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;" rel="noreferrer">@mtijink</a> if you plan on contributing more to the app, you should consider asking for a KDE developer account, once you have some patches like this one in :)</p>
<p>Sorry for having code reviews like this one hanging for long recently, I've changed roles at work and I'm still getting used to it. Thankfully there is people taking care of the reviews without me :D</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>mpris-data-per-player</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8942" rel="noreferrer">https://phabricator.kde.org/D8942</a></div></div><br /><div><strong>To: </strong>mtijink, KDE Connect, thomasp<br /><strong>Cc: </strong>albertvaka, apol, thomasp<br /></div>