<table><tr><td style="">ltoscano resigned from this revision.<br />ltoscano 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/D7130" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I still disagree in the way this is done, as it means manually periodically updating the information and it's definitely not the right thing to do.</p>
<p>That said, if <a href="https://phabricator.kde.org/p/elvisangelaccio/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;" rel="noreferrer">@elvisangelaccio</a> approves this, it can go in as temporary fix but the next update should work on using the appstream library to get those information from appstream files. Or any other solutions which does not involve duplicating the information.</p>
<p>I'm removing my vote but please wait for Elvis.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>BRANCH</strong><div><div>mydev</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7130" rel="noreferrer">https://phabricator.kde.org/D7130</a></div></div><br /><div><strong>To: </strong>gregormi, ltoscano, elvisangelaccio, KDE Applications, davidedmundson, ngraham<br /><strong>Cc: </strong>ngraham, ltoscano, Frameworks<br /></div>