<div dir="ltr"><div>On Thu, Jul 31, 2014 at 1:49 PM, Srikanth Koneru <span dir="ltr"><<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div dir="ltr">But the bottle neck is admin time availability.</div></blockquote><div><br></div><div>I grant you that admin time is a problem, but additionally, the fact that admins have to second guess and redo most reviews is also kind of a problem.</div>

<div><br></div><div>I just went and looked at four "approved" themes in the admin queue. Two of them had issues that I noticed immediately as being problems. Big-code-issues that prevent me from going live with them; not just look and feel type things. The other two I marked live. Now, small sample, I grant you, but if this holds true throughout this queue, then that's about 40 themes I'll have to return-to-sender instead of marking live. </div>

<div><br></div><div>I can see why the current admins would find this a bit frustrating.</div></div><div class="gmail_extra"><br></div><div class="gmail_extra">What can help make it such that reviewers are actually doing the entire review before approving themes? Or is there a focus problem? In the cases I've seen, it's been functionality issues that are the stoppers at this stage, not necessarily look-and-feel issues. Most reviews I've been reading over the last 10 minutes pointed out menu problems, CSS problems, things like the front-page functionality not working properly, but missed (to me) big obvious ones like saving defaults to the database or requiring the use of a plugin. This is the sort of things the admins find and I grant you that you have to look at the code for them, but I don't think the admins should need to be second-reviewing everybody's reviews here, but that is what is happening and why it takes so long. </div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Should we be assigning reviewers in pairs? Two reviewers per ticket? I dunno.</div><div class="gmail_extra"><br clear="all"><div>-Otto</div>
<div class="gmail_quote"><br></div><div class="gmail_quote"><br></div></div></div>