<div dir="ltr">In general, when finding issues, you should note them in the ticket, and allow the developer time to resolve the issues and re-upload. The workflow is designed to allow multiple revisions to proceed in a single ticket, as necessary, until ultimate Theme approval.<br>
<br>You should only close a ticket for egregious issues, issues that cannot feasibly be resolved in a timely manner, or due to lack of developer response. (Note: I generally close tickets after one week with no developer response to reviewer comments.)<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 5, 2014 at 3:17 PM, rabin shrestha <span dir="ltr"><<a href="mailto:sun_ravi90@yahoo.com" target="_blank">sun_ravi90@yahoo.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-size:12pt;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif"><div>
Hi,</div><div><br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif">I have a query. If we find any issues (Notices, Warnings or any required issues) in previously approved theme while reviewing for it's version update. Should we resolve it as not approved or should we keep the ticket open and comment on the ticket for calling theme author to fix the issue and upload the new version.</div>
<div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif">or should we point out issues, approve the theme and ask theme author to fix it in
 next version. What will be the best approach<br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif"><br></div>
<div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif">Regards,</div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif">
Rabin<br></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif"><br></div></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>