[theme-reviewers] recomended vs required

Mon Oct 13 21:32:24 UTC 2014

sound like a good plan.
thank you so much for all input.

- david.
On Oct 14, 2014 4:29 AM, "Tammie Lister" <karmatosed at gmail.com> wrote:

>  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> 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>
>> 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>
>>> 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> 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
>>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>>
>>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>
>>>>
>>>>   --
>>>> Thanks
>>>> Tammie Lister
>>>>
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing list
>>>> 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
>>>
>>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
>
> _______________________________________________
> theme-reviewers mailing listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
> --
> Thanks
> Tammie Lister
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20141014/a1e29c58/attachment-0001.html>