<div dir="ltr">Most people will leave the ticket open and give the developer an opportunity to resubmit. Personally, I only close tickets when the developer isn't making an effort to actually test issues before resubmitting, as I don't think it's my job as a reviewer to do a developer's QA or bug testing.<div>

<br></div><div>Honestly this isn't much of an issue since we've eliminated waiting times for reviews. If a ticket is closed as not approved, when they open a new ticket it will get reviewed in about the same amount of time as it would take if the ticket was still open, possibly faster.</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 28, 2013 at 6:36 AM, Swagata Das <span dir="ltr"><<a href="mailto:swagata_das@siteobtain.com" target="_blank">swagata_das@siteobtain.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>



<div style="PADDING-LEFT:10px;PADDING-RIGHT:10px;PADDING-TOP:15px" name="Compose message area">
<div><font face="Arial">Hello all and my senior reviewers, after joining 
reviewer team over these days I followed many tickets to learn more. In many 
cases I noticed some of us while reviewing a theme ain't giving any chances 
to the developer for fixing issues by directly setting it as "not approved" and 
that's what I did once in my first time too. My opinion is that we should give a 
chance to the developer for fixing those issues instead of setting 
the resolution as "not approved" in first comment. Looking for your 
opinion...</font></div></div>
<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>
<br></blockquote></div><br></div>