Hi Chip,<br><br>I am getting email for comments. I hope it is fine now. I will complete this review and ask for another one. Let's see at that time. I will let you know. Thanks a lot.<br><br>Regards,<br>Sakin<br><br><div class="gmail_quote">
On Wed, Aug 1, 2012 at 9:49 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Odd on both counts, as I got an email notification in both cases. Can you please both try posting comments on your assigned tickets, and let me know if you do or do not get an email notification of the comment?<div><br></div>
<div>Thanks!</div><span class="HOEnZb"><font color="#888888"><div><br></div></font></span><div><span class="HOEnZb"><font color="#888888">Chip</font></span><div><div class="h5"><br><br><div class="gmail_quote">On Wed, Aug 1, 2012 at 10:53 AM, Sakin Shrestha <span dir="ltr"><<a href="mailto:info@catchinternet.com" target="_blank">info@catchinternet.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yes Chip, I just checked in and my email address is also saved. But I didn't get any message.<div><div>
<br><br><div class="gmail_quote">On Wed, Aug 1, 2012 at 9:35 PM, Chandra Maharzan <span dir="ltr"><<a href="mailto:maharzan@gmail.com" target="_blank">maharzan@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks Chip. I saw me assigned. FYI, I have the email in the prefs but<br>
didn't get any notification about getting assigned. I checked the trac<br>
directly.<br>
<div><div><br>
On Wed, Aug 1, 2012 at 9:30 PM, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>> wrote:<br>
> Be sure to add your email address to your Theme-Trac profile:<br>
> <a href="http://themes.trac.wordpress.org/prefs" target="_blank">http://themes.trac.wordpress.org/prefs</a><br>
><br>
><br>
> On Wed, Aug 1, 2012 at 10:39 AM, Sakin Shrestha <<a href="mailto:info@catchinternet.com" target="_blank">info@catchinternet.com</a>><br>
> wrote:<br>
>><br>
>> Why I don't get any notification. Is there anything that I need to do to<br>
>> get notification.<br>
>><br>
>><br>
>> On Wed, Aug 1, 2012 at 9:21 PM, Chandra Maharzan <<a href="mailto:maharzan@gmail.com" target="_blank">maharzan@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> Sakin, I see you already have a theme in your plate. :)<br>
>>><br>
>>> On Wed, Aug 1, 2012 at 9:15 PM, Sakin Shrestha <<a href="mailto:info@catchinternet.com" target="_blank">info@catchinternet.com</a>><br>
>>> wrote:<br>
>>> > Also I requested at the Queue and haven't got. My username is:<br>
>>> > catchthemes<br>
>>> ><br>
>>> ><br>
>>> > On Wed, Aug 1, 2012 at 9:09 PM, Chandra Maharzan <<a href="mailto:maharzan@gmail.com" target="_blank">maharzan@gmail.com</a>><br>
>>> > wrote:<br>
>>> >><br>
>>> >> Cool. Who assigns a ticket? I request over at the Queue and haven't<br>
>>> >> got anything yet. Would love to start. Thanks.<br>
>>> >><br>
>>> >> On Wed, Aug 1, 2012 at 9:04 PM, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>><br>
>>> >> wrote:<br>
>>> >> > Generally speaking, no. I can't think of anything. Themes are<br>
>>> >> > expected<br>
>>> >> > to<br>
>>> >> > conform to the current Guidelines at the time of submission. The<br>
>>> >> > reason<br>
>>> >> > that<br>
>>> >> > Theme Name is "grandfathered" is that changing the Theme Name<br>
>>> >> > changes<br>
>>> >> > the<br>
>>> >> > Theme Slug, which in turn breaks automatic update notification for<br>
>>> >> > end<br>
>>> >> > users.<br>
>>> >> ><br>
>>> >> > I would recommend to just go ahead and review some Themes. If you<br>
>>> >> > observe a<br>
>>> >> > "required" issue, note it in the review. If the developer pushes<br>
>>> >> > back on<br>
>>> >> > any<br>
>>> >> > of the review comments, just try to work it out in-ticket. And if<br>
>>> >> > you<br>
>>> >> > need<br>
>>> >> > assistance, don't hesitate to ask another reviewer or an admin to<br>
>>> >> > offer<br>
>>> >> > a<br>
>>> >> > second opinion on the ticket.<br>
>>> >> ><br>
>>> >> > Thanks,<br>
>>> >> ><br>
>>> >> > Chip<br>
>>> >> ><br>
>>> >> ><br>
>>> >> > On Wed, Aug 1, 2012 at 10:09 AM, Chandra Maharzan<br>
>>> >> > <<a href="mailto:maharzan@gmail.com" target="_blank">maharzan@gmail.com</a>><br>
>>> >> > wrote:<br>
>>> >> >><br>
>>> >> >> So, is there anything else that can skip the guidelines apart from<br>
>>> >> >> name for already hosted themes? I am interested in reviewing themes<br>
>>> >> >> so<br>
>>> >> >> want to understand this better. Thanks for your responses Chip.<br>
>>> >> >><br>
>>> >> >> On Wed, Aug 1, 2012 at 8:49 PM, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>><br>
>>> >> >> wrote:<br>
>>> >> >> > "For obvious reasons, we are quite reluctant to ask a Theme<br>
>>> >> >> > already<br>
>>> >> >> > hosted<br>
>>> >> >> > in the WPORG repository to change its name."<br>
>>> >> >> ><br>
>>> >> >> > I'm not sure how that statement can be misconstrued to mean that<br>
>>> >> >> > we<br>
>>> >> >> > would<br>
>>> >> >> > allow a Theme to use deprecated functions.<br>
>>> >> >> ><br>
>>> >> >> > Chip<br>
>>> >> >> ><br>
>>> >> >> ><br>
>>> >> >> > On Wed, Aug 1, 2012 at 10:01 AM, Chandra Maharzan<br>
>>> >> >> > <<a href="mailto:maharzan@gmail.com" target="_blank">maharzan@gmail.com</a>><br>
>>> >> >> > wrote:<br>
>>> >> >> >><br>
>>> >> >> >> I am emailing because I saw this (new version) being approved<br>
>>> >> >> >> just 3<br>
>>> >> >> >> days ago. Does that mean, when the theme is updated, you guys<br>
>>> >> >> >> don't<br>
>>> >> >> >> review the all the guidelines ? only the difference ? even if<br>
>>> >> >> >> uses<br>
>>> >> >> >> deprecated functions and may not work with new WP versions ?<br>
>>> >> >> >><br>
>>> >> >> >> On Wed, Aug 1, 2012 at 8:41 PM, Chip Bennett<br>
>>> >> >> >> <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>><br>
>>> >> >> >> wrote:<br>
>>> >> >> >> > That Theme was first approved in 2009, and therefore predates<br>
>>> >> >> >> > the<br>
>>> >> >> >> > current<br>
>>> >> >> >> > Theme Review guidelines, including the Theme Name requirement.<br>
>>> >> >> >> ><br>
>>> >> >> >> > For obvious reasons, we are quite reluctant to ask a Theme<br>
>>> >> >> >> > already<br>
>>> >> >> >> > hosted in<br>
>>> >> >> >> > the WPORG repository to change its name.<br>
>>> >> >> >> ><br>
>>> >> >> >> > Chip<br>
>>> >> >> >> ><br>
>>> >> >> >> > On Wed, Aug 1, 2012 at 9:33 AM, Chandra Maharzan<br>
>>> >> >> >> > <<a href="mailto:maharzan@gmail.com" target="_blank">maharzan@gmail.com</a>><br>
>>> >> >> >> > wrote:<br>
>>> >> >> >> >><br>
>>> >> >> >> >> Hi Guys,<br>
>>> >> >> >> >><br>
>>> >> >> >> >> I have been looking into the theme review guidelines and been<br>
>>> >> >> >> >> finding<br>
>>> >> >> >> >> few things such as the Twenty Eleven error. :) Now I found<br>
>>> >> >> >> >> another<br>
>>> >> >> >> >> one. iStudio Theme:<br>
>>> >> >> >> >> <a href="http://wordpress.org/extend/themes/istudio-theme" target="_blank">http://wordpress.org/extend/themes/istudio-theme</a><br>
>>> >> >> >> >><br>
>>> >> >> >> >> While the guidelines say you can't add a theme in the name:<br>
>>> >> >> >> >> <a href="http://codex.wordpress.org/Theme_Review#Theme_Name" target="_blank">http://codex.wordpress.org/Theme_Review#Theme_Name</a>, this<br>
>>> >> >> >> >> theme<br>
>>> >> >> >> >> still<br>
>>> >> >> >> >> has it and I saw it was just approved in the Most recently<br>
>>> >> >> >> >> updated<br>
>>> >> >> >> >> themes. Perhaps, this can go in their next update. Just a<br>
>>> >> >> >> >> heads<br>
>>> >> >> >> >> up<br>
>>> >> >> >> >> to<br>
>>> >> >> >> >> the Review Team.<br>
>>> >> >> >> >><br>
>>> >> >> >> >> Thanks,<br>
>>> >> >> >> >> Chandra<br>
>>> >> >> >> >><br>
>>> >> >> >> >> --<br>
>>> >> >> >> >> cmans<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>
>>> >> >> >> ><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>
>>> >> >> >><br>
>>> >> >> >><br>
>>> >> >> >><br>
>>> >> >> >> --<br>
>>> >> >> >> cmans<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>
>>> >> >> ><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>
>>> >> >><br>
>>> >> >><br>
>>> >> >><br>
>>> >> >> --<br>
>>> >> >> cmans<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>
>>> >> ><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>
>>> >><br>
>>> >><br>
>>> >><br>
>>> >> --<br>
>>> >> cmans<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>
>>> ><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>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> cmans<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>
>><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>
><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>
<br>
<br>
<br>
--<br>
cmans<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>
</div></div></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>