Devin - <br><br>Yes, your review should be enough to get the author working on revisions ...<br><br>You might consider reading over some of the other closed tickets with reviews to get some additional ideas on how we have been presenting points.<br>
<br>If you believe the theme should be marked "not-accepted" go ahead; I generally check Trac versus Extend to make sure they are in sync most days, this includes making sure the tickets marked "not-accepted" are marked as old in extend; and, tickets noted as "suggest-approval" are handled properly as well. This in turn will advise the theme author their theme has been reviewed and its status.<br>
<br>I will leave your current review ticket as is until you have marked it "not-accepted" as you suggested.<br><br><br>Cais.<br><br><br><div class="gmail_quote">On Sun, Aug 1, 2010 at 2:44 PM, Devin Price <span dir="ltr"><<a href="mailto:devin@wptheming.com">devin@wptheming.com</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 bgcolor="#ffffff" text="#000000">
I did my first review of a theme:
<a href="http://themes.trac.wordpress.org/ticket/414" target="_blank">http://themes.trac.wordpress.org/ticket/414</a>. I'm not sure I caught
everything, but should be enough for the author to work on. Should I
just resolve as "Not Accepted"?<br>
<br>
Will the theme author automatically get notified? If anyone wants to
e-mail me directly about how to improve my reviews, I'd appreciate it
<a href="mailto:devin@wptheming.com" target="_blank">devin@wptheming.com</a>. I'd love to knock through a few of these so I can
eventually get my theme reviewed. :)<br>
<div>
<p>Devin Price<a href="http://www.wptheming.com" target="_blank"><br>
</a></p>
</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>