[theme-reviewers] recomended vs required
Tammie Lister
karmatosed at gmail.com
Mon Oct 13 21:29:51 UTC 2014
David, your theme has had a new version uploaded. It is now marked as
such and the reviewer attached to your new theme. No need for any
haircuts or this thread to go on anymore.
How about we all work together rather than creating long threads? If
something needs talking about in the ticket, lets do that. If someone
thinks there is a better way to work, great we can always refine. We are
all human, we all need to improve us and our processes. Anyone is
welcome to come to the chat tomorrow and talk about how the process
could be better. Anyone can talk about that at any time.
What I think isn't great is creating this long thread like this in the
tone it has devolved into in places. Lets all remember we are in this
together. Lets remember there are humans at the end of the computers.
Now lets all get on with being awesome - sound like a good plan?
Thanks
Tammie
On 13/10/2014 22:23, David Chandra wrote:
> @ Srikanth Koneru:
> I'm not a theme reviewer and i don't really understand theme review
> workflow.
> But it seems like a good plan.
>
> the reviewer to my theme actually reopen the ticket and it is kicked
> back to original reviewer.
> I'm perfectly fine with that.
>
> But, in my understanding, the theme reviewer who review my theme update
> and give suggestion about screenshot, already reviewing the update.
> (that's why the reviewer reopen and give suggestion to the theme, right)
>
> Maybe if I just update the screenshot and put several inline docs, it
> will get approved and go live.
> I can see that this can be frustrating to theme author.
>
> David.
>
> /Maybe I just need a hair cut. Do you think i need a haircut?/
>
>
>
>
> On Tue, Oct 14, 2014 at 4:08 AM, Srikanth Koneru <tskk79 at gmail.com
> <mailto:tskk79 at gmail.com>> wrote:
>
> Maybe we should not reopen tickets unless its license and/or
> security issue, small issues even a few can easily fixed
> downstream. Reopening will only add to reviewer workload, it will
> be kicked back to original reviewer and if that reviewer is
> missing, it will only add to theme author's frustration.
>
> On Tue, Oct 14, 2014 at 2:29 AM, David Chandra
> <david at shellcreeper.com <mailto:david at shellcreeper.com>> wrote:
>
> Thank you Tammie,
> I understand that the recommendation by the theme reviewer is
> for the best.
> I do understand that personal/professional knowledge vs theme
> guidelines can sometimes in a blurry line.
>
> once i got a reviewer said:
> It's required for all functions in functions.php to have
> function_exist check.
> (which is not even in guidelines)
>
> I wish to theme reviewer to better understand the guidelines.
> so they can better inform to theme author about their theme,
> which are "recommended" which are "required" and maybe their
> personal
> "opinion" and/or "suggestion" to make the theme better.
>
> Of course, the theme authors can receive their suggestion in
> better perspective.
>
> -- David.
>
>
>
>
> On Tue, Oct 14, 2014 at 3:32 AM, Tammie Lister
> <karmatosed at gmail.com <mailto:karmatosed at gmail.com>> wrote:
>
> Nobody is trying to single you out, nobody is making an
> example of you, nobody is witch hunting anyone. Everyone
> just wants to make your theme the best it can be and work
> with you to get your theme live. Just consider this when
> interacting and don't assume someone's actions. Asking and
> having an open discussion with a person always works better.
>
> Thanks
> Tammie
>
>
> On 13/10/2014 21:09, David Chandra wrote:
>> Thank you Justin.
>>
>> and I'm sorry to all if my comment about "witch hunt
>> target" is a too "strong".
>> I didn't mean it that way. I'm truly understand what the
>> reviewer suggested is good. very good.
>>
>> I responded that way because i feel, maybe as example:
>>
>> I want to create a passport, and I completed all the
>> legal document,
>> and finally after hours and hours of waiting in line I
>> take photo shoot,
>> and all I need is to wait for the passport.
>> But suddenly they said that
>> "Sir, you need to cut your hair, i think your passport
>> photo will look better"
>> and i respond: "No thanks, I think i look fine."
>> and they respond "Are you sure you want to look like this
>> in your photo? You need to cut your hair. Your photo is
>> going to be there for 5 years."
>>
>> of course my respond would be "what the..." i need to
>> clarify about the passport registration procedure.
>>
>> -- David.
>>
>>
>>
>>
>>
>> On Tue, Oct 14, 2014 at 2:53 AM, Justin Tadlock
>> <justin at justintadlock.com
>> <mailto:justin at justintadlock.com>> wrote:
>>
>> do theme reviewer now start to not approving
>> themes because not doing the recomended even
>> already do all required in guidelines?
>>
>> Recommended guidelines are only recommended. It's
>> often a good idea to do them, but you are not
>> required to do so.
>>
>> if the theme already pass all the required in the
>> guidelines do theme reviewer need to make theme
>> author to do extra recommended guide?
>>
>> It'd be great if reviewers would mention these
>> things, but there's no reason a reviewer should hold
>> a theme back from being approved for things marked
>> recommended.
>>
>> Also, not all guidelines that are recommended are
>> good for every theme. For example, the custom
>> background feature is a recommended feature, but it
>> doesn't make sense to use that feature in all
>> themes. In fact, it can be a downright horrible
>> feature to use with some theme designs. Basically,
>> recommended items are things we want you to be aware
>> of. We want you to think about whether these
>> additions would be a good thing for your theme.
>>
>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> <mailto:theme-reviewers at lists.wordpress.org>
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org <mailto:theme-reviewers at lists.wordpress.org>
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
> --
> Thanks
> Tammie Lister
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> <mailto:theme-reviewers at lists.wordpress.org>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> <mailto:theme-reviewers at lists.wordpress.org>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> <mailto:theme-reviewers at lists.wordpress.org>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
--
Thanks
Tammie Lister
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20141013/c452b59f/attachment-0001.html>
More information about the theme-reviewers
mailing list