<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style id="owaParaStyle" type="text/css">P {margin-top:0;margin-bottom:0;}</style>
</head>
<body ocsi="0" fpstyle="1">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">I know you guys are trying to speed up the process and that trivial errors should be overlooked. In this case though the theme has some security vulnerabilities, and the functions
are starting with inconsistent slugs. I didn't go through every file or even test the theme in WordPress, all I did was look at a few files of the theme in the trac. I will be leaving a comment on that ticket for the theme reviewer so they can go back and
have a look. Thanks for your help.<br>
<div style="font-family: Times New Roman; color: rgb(0, 0, 0); font-size: 16px;">
<hr tabindex="-1">
<div style="direction: ltr;" id="divRpF235606"><font face="Tahoma" size="2" color="#000000"><b>From:</b> theme-reviewers-bounces@lists.wordpress.org [theme-reviewers-bounces@lists.wordpress.org] on behalf of Edward Caissie [edward.caissie@gmail.com]<br>
<b>Sent:</b> Friday, December 16, 2011 8:46 AM<br>
<b>To:</b> theme-reviewers@lists.wordpress.org<br>
<b>Subject:</b> Re: [theme-reviewers] A Quick Question<br>
</font><br>
</div>
<div></div>
<div>A little of both ...<br>
<br>
As we are trying to expedite the process, at least in the short term, I would expect there to be more items missed especially in themes that are resolved as "not-approved" but if there are themes that have been resolved as approved and there are "non-trivial"
issues found after the fact then whomever finds them should be commenting on the appropriate ticket.<br>
<br>
You can also drop me a note specifically if you feel the issue requires immediate attention as a show-stopper before being approved into the repository as I go through the entire Extend admin queue at least once a week to make sure Trac and Extend are in sync.<br>
<br>
The idea is not that you would be stepping on another reviewers toes, but that we are working as a team and sometimes a reviewer just misses an item, for whatever reason, but we should still have each others back all the same. It is much better to have a theme
corrected before it goes live to Extend as the only way to "fix" any issues is by a very quick update by the author (special circumstances immediate review as necessary) and/or temporary suspension of the theme as there is no "un-approve" method once the theme
is live.<br>
<br>
<br clear="all">
Cais.<br>
<br>
<br>
<div class="gmail_quote">On Fri, Dec 16, 2011 at 6:50 AM, Rankin, Matthew W. (Student)
<span dir="ltr"><<a href="mailto:mrankin@my.ccsu.edu" target="_blank">mrankin@my.ccsu.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div>
<div style="direction: ltr; font-family: Tahoma; color: rgb(0, 0, 0); font-size: 10pt;">
I see that now after one review everyone gets "full reviewer" status ( for those of you who didn't know that please see this post
<a href="http://make.wordpress.org/themes/2011/12/15/getting-the-review-queue-back-on-track/#comment-15374" target="_blank">
http://make.wordpress.org/themes/2011/12/15/getting-the-review-queue-back-on-track/#comment-15374</a> ). I think this is a great idea and will really speed up the review process. Now for my question, I was playing with the trac software just getting a better
feel for it now that I have "full reviewer" status and when I viewed the timeline there is a theme that was recently marked as approved that has some errors. I was wondering what the policy is on stepping in on others reviews? I don't want to be rude and step
on other people's toes, but I don't want to let themes with errors in either. Can someone please tell me what to do? Should I report it to an admin? Should I leave a comment on the ticket? Should I let it go? Any advice here is appreciated.<br>
</div>
</div>
<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
</div>
</div>
</body>
</html>