<div dir="ltr"><div><div>If approved themes are not-approved, next versions end up in that queue.<br></div>Not sure, but i guess they should fix them before they are approved like any other not-approved/new tickets.<br></div>
You are setting the trac on fire :)<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Feb 24, 2013 at 2:53 PM, Greg Priday <span dir="ltr">&lt;<a href="mailto:greg@siteorigin.com" target="_blank">greg@siteorigin.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I just picked up this ticket <a href="http://themes.trac.wordpress.org/ticket/10913" target="_blank">http://themes.trac.wordpress.org/ticket/10913</a><br>

<br>
After giving it a full review I noticed that it&#39;s already live in the<br>
directory. For some reason it landed up in the &quot;Previously Reviewed,<br>
Not Approved&quot; queue.<br>
<br>
Should I approve the theme based on a diff review or require that the<br>
author fix the required issues I found?<br>
<br>
--<br>
I make free WordPress themes<br>
<a href="http://siteorigin.com" target="_blank">http://siteorigin.com</a><br>
<a href="http://facebook.com/SiteOrigin" target="_blank">http://facebook.com/SiteOrigin</a><br>
<a href="http://twitter.com/SiteOrigin" target="_blank">http://twitter.com/SiteOrigin</a><br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</blockquote></div><br></div>