Chip, isn't it 'good to have' to keep tickets open? In my opinion most themes need approx. 3-4 iterations to get in and given the stats, that might take few months even though fixes might take a few hours.<div>
<br></div><div>IMO theme authors that prioritize theme submissions should have a fast lane open when the feedback could apparently be resolved in a few hours. This is in case reviewers conduct a complete review and not a quick look only.<br clear="all">
<div><br>Mario Peshev<br>WordPress Engineer, Open Source Consultant<br><a href="http://www.linkedin.com/in/mpeshev" target="_blank" class="vt-p">http://www.linkedin.com/in/mpeshev</a><br><div><a href="http://me.peshev.net/" target="_blank" class="vt-p">http://me.peshev.net</a></div>
<div><br></div></div>
<br><br><div class="gmail_quote">On Mon, Jan 7, 2013 at 4:11 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank" class="vt-p">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>This is especially important for previously approved (Priority #1) tickets, since such tickets already receive an expedited, diff-only review.</div><div><br></div><div>Two, we have a great system in place, that allows a reviewer to hold a ticket open if only minor issues need to be addressed, such that once a revision is submitted, the reviewer can continue the current review in the new ticket. If the previous ticket is still open, the new ticket should be left for the reviewer of the previous ticket, so that the previous review may be continued.</div>
<div><br></div><div>I had two such tickets over the weekend: both had previous-ticket "required" issues that were not addressed. I left the tickets open to allow for a review continuation. Both developers uploaded revisions, but when I went in this morning to take the tickets, both had been assigned, reviewed, and closed. Unfortunately, in both cases, the review failed to indicate whether previous-ticket "required" issues had been addressed.</div>
<div></div></blockquote></div><br></div>