True enough, "suggest-approval" is fairly redundant compared to "approved" ... although I would suggest simply not using that particular resolution for the time being. My idea being: how does removing a resolution affect tickets already using it?<br>
<br>IF there is no effect, I imagine once someone with the appropriate permission set has time they can remove it. If it does have an effect then just leave it alone until those affects can be sorted out and addressed properly.<br>
<br><br>Cais.<br><br><div class="gmail_quote">On Thu, Oct 14, 2010 at 2:35 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</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;">
I'm under the assumption at this point that, unless we hear otherwise from Otto or someone, that the original Trainee Workflow idea isn't feasible. So, under that assumption, we'd have no need for "suggest-approval" or "suggest-not-approved" as ticket resolutions.<div>
<br></div><div>For the time being, having the Trainee indicate one of those two resolutions, and then having a Reviewer resolve/close the ticket accordingly, will probably work about the best we can hope for.</div><div><br>
</div><div>In which case, we can eliminate "suggest-approval" as a ticket Resolution.</div><div><br></div><div><font color="#888888">Chip</font><div><div></div><div class="h5"><br><br><div class="gmail_quote">On Thu, Oct 14, 2010 at 1:32 PM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.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;">Yes, in the most minimalist fashion, resolutions of: Yes, No, and Maybe are all that we need, but we have also recently put forward, as in another suggestion of yours to use "suggest-approval" and "suggest-not-approval" for the Theme Reviewer "Trainees" which although the balance of the idea is not well supported by our current tools, those resolutions still make sense to use under their "new" proposed meanings.<br>
<br>Also, just as a reminder for those not familiar with Trac, all resolutions, no matter their label, close the ticket.<br><br><br>Cais.<div><div></div><div><br><br><div class="gmail_quote">On Thu, Oct 14, 2010 at 12:45 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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;">Okay, then, here's what I propose:<div><br></div><div>1) Eliminate "needs-additional-review"</div>
<div>2) Eliminate "suggest-approval"</div><div><br></div><div>We have no need for either one:</div><div>
<ul><li>"approved" - Theme passes review, and ticket is closed</li><li>"not-approved" - Theme does not pass review, and ticket is closed</li><li>"closed-newer-version-uploaded" - ticket is closed, and a newer version of the Theme is reviewed</li>
</ul></div><div>That covers all of our bases right now.</div><div><br></div><font color="#888888"><div>Chip</div></font><div><div></div><div><div><br><div class="gmail_quote">On Thu, Oct 14, 2010 at 11:37 AM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.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;">"approved with comments" and related ideas were the impetus to the creation of "needs-additional-review" and IIRC "suggest-approval" resolutions ... if we are going to add more resolutions we need to understand what the existing ones are to be used for.<br>
<br>So as you think I have a problem or issue with this approach, I would rather write I have concerns in continually going forward without looking at where we came from.<br><br>We should be solidifying our basics before building on them. As I mentioned before, the current resolutions need to be better defined so reviewers and end-users alike understand what they are for. If that means we need to add more afterward I am fine with that, too, but currently we have resolutions that were meant to cover your "original subject" to my understanding.<br>
<br>If our current resolutions are not sufficient, obviously we can add more, but we should define the existing ones first is essentially what I am putting forward.<br><br><br>Cais.<div><div></div><div><br><br>
<div class="gmail_quote">On Thu, Oct 14, 2010 at 12:23 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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 class="gmail_quote">So, we circle back to the original subject: the use of "approved with comments".</div>
<div class="gmail_quote"><br></div><div class="gmail_quote">I'm trying to understand your disagreement with this method. To me, it is a reasonable compromise between approving generally good Themes, while also moving Theme Developers toward increased conformance to the Guidelines.</div>
<div class="gmail_quote"><br></div><div class="gmail_quote">So, can you help me understand your problems/issues with this approach? Is it the approach itself, or is it the idea of formalizing it?</div><div class="gmail_quote">
<br></div><font color="#888888"><div class="gmail_quote">Chip</div><div class="gmail_quote"><br></div></font><div class="gmail_quote"><div><div></div><div>On Thu, Oct 14, 2010 at 11:15 AM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div><div></div><div><div><div class="gmail_quote">On Thu, Oct 14, 2010 at 11:53 AM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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;">
AFAIK, either we enforce them, or we don't.</blockquote></div><br></div>If that is our method then there are no "minor" issues ... and quite simply that is the crux of the matter. <br><br>The guideline needs to be strongly adhered to with a "black or white" premise but that does not preclude reasonable exceptions and as you are want to describe "selective enforcement", or in my mind reasonable interpretations of the Theme Review page(s) to meet the requirements as they are expected to be met.<br>
<br>Would I ignore the current license requirement as you are quoting, in a word: Yes. Would I ignore the complete lack of any sort of GPL-compliance declaration, again in a word: No. If the author has chosen another method to declare the theme GPL compliant that resembles the quote above, then I would likely accept it and most likely suggest they use what the Theme Review page(s) state should be used (at this time). We have already decided that will be changing to something much more "blank and white" in the (near) future.<br>
<br>Rather than continually re-hashing this particular point we should be addressing the future requirements of the GPL compatible license declaration(s) and putting that forward.<br><br>Also to the CSS requirements ... once "FixPress" is not required to have a standard default WordPress installation using the most current Theme Unit Test data pass the validation test(s) I will be happy to re-consider setting a resolution of "not-approved" based on minor CSS issues, until then I will remain using, as you like to refer to it, "selective enforcement".<br>
<br><br>Cais.<br>
<br></div></div><div>_______________________________________________<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></div></blockquote></div><br>
<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><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><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><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>
<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>