<table><tr><td style="">huftis requested changes to this revision.<br />huftis 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/D17739">View Revision</a></tr></table><br /><div><div><p>In <tt style="background: #ebebeb; font-size: 13px;">poToDataset.py</tt>, only translated strings should be included. Currently, if a translator translates ‘foo’ to ‘bar’, waits until the JSON file is regenerated, changes their mind and deletes or fuzzies the translation (“I don’t think ‘bar’ is the correct translation for ‘foo’ after all, but I’m not sure what <em>is</em> the correct translation yet”), the JSON file is stuck with ‘bar’ as the translation.</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/D17739#inline-101721">View Inline</a><span style="color: #4b4d51; font-weight: bold;">poToDataset.py:49</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">else</span><span class="p">:</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span style="color: #aa4000">print</span><span class="p">(</span><span class="n">word</span><span class="p">,</span> <span style="color: #766510">"not found in json file"</span><span class="p">)</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">This doesn’t delete old, untranslated/fuzzy entries. It should.</p></div></div></div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17739">https://phabricator.kde.org/D17739</a></div></div><br /><div><strong>To: </strong>jjazeix, GCompris, Localization, huftis<br /><strong>Cc: </strong>pino, huftis, aacid, kde-edu, harrymecwan, ganeshredcobra, nityanandkumar, echarruau, rahulyadav, narvaez, scagarwal, apol, timotheegiet, hkaelberer, jjazeix, bcoudoin<br /></div>