[theme-reviewers] New review workflow

Chip Bennett chip at chipbennett.net
Wed May 22 18:59:51 UTC 2013


Please verify that you have added your email address to your Theme-Trac
user preferences:
http://themes.trac.wordpress.org/prefs


On Wed, May 22, 2013 at 2:56 PM, Srikanth Koneru <tskk79 at gmail.com> wrote:

> I get all emails when i am the theme author, i.e. new upload/reviewer
> accepting the ticket/reviewer comments/closing the ticket/theme live
> notification.
> I don't get the emails when i am the reviewer. i.e. when theme author
> replies/comments inticket.
>
>
> On Thu, May 23, 2013 at 12:20 AM, Chip Bennett <chip at chipbennett.net>wrote:
>
>> Do you get other emails, and are just not getting comment notification
>> emails? Or do you get no Theme-Trac emails at all?
>>
>>
>> On Wed, May 22, 2013 at 2:43 PM, Srikanth Koneru <tskk79 at gmail.com>wrote:
>>
>>> Nice, did a quick review.
>>> For some reason i don't get email when the theme author replies, is that
>>> normal or do i need to check some box?
>>>
>>>
>>> On Wed, May 22, 2013 at 11:45 PM, Jose Castaneda <jomcastaneda at gmail.com
>>> > wrote:
>>>
>>>> Huge thanks to both you and Andrew Nacin, Otto. Looking forward to
>>>> doing my first review under the new system. Hopefully tomorrow. :)
>>>>
>>>>
>>>> On Wed, May 22, 2013 at 11:12 AM, Emil Uzelac <emil at uzelac.me> wrote:
>>>>
>>>>> It sure does help, thanks :)
>>>>>
>>>>> Emil
>>>>>
>>>>>
>>>>> On Wed, May 22, 2013 at 1:06 PM, Otto <otto at ottodestruct.com> wrote:
>>>>>
>>>>>> You all may have noticed that the review process in Trac is changing.
>>>>>> This is a work-in-progress, in an attempt to streamline the process
>>>>>> and make it a little more obvious and clear. Additionally, we will
>>>>>> (eventually) be automating the "push-to-live" process to attempt to
>>>>>> make that faster and to hopefully eliminate the 1 week delay emails on
>>>>>> this list.
>>>>>>
>>>>>> You can thank/blame Andrew Nacin for all of this. He knows the ins and
>>>>>> outs of Trac far better than I do. :)
>>>>>>
>>>>>> Here's a brief overview of the process:
>>>>>>
>>>>>> Reviewers: You will have the ability to go to a ticket and choose one
>>>>>> of two things:
>>>>>>  * review
>>>>>>  * resolve as closed-newer-version-uploaded
>>>>>>
>>>>>> The idea here is that you pick a ticket to review, then change the
>>>>>> status to "review". This will mark the ticket as being owned by you
>>>>>> and say that you are in the process of reviewing it.
>>>>>>
>>>>>> Once you're done, you can post your review, and you have the ability
>>>>>> to:
>>>>>>  * resolve as not-approved
>>>>>>  * resolve as closed-newer-version-uploaded
>>>>>>  * approve
>>>>>>
>>>>>> The first two are obvious, but the "approve" is changing. Now it will
>>>>>> no longer close the ticket. Instead, it will mark the ticket as
>>>>>> approved, and go into a separate queue for the admins. This queue will
>>>>>> let the admins, from Trac, choose to either "reopen" the ticket,
>>>>>> leaving comments and explaining what more needs to be done, or they
>>>>>> can mark the theme as ready for going live.
>>>>>>
>>>>>> Marking the theme as ready for live will, eventually, do the process
>>>>>> automatically and make the theme live on WordPress.org/themes. Still
>>>>>> working on that bit. :)
>>>>>>
>>>>>> The admins can also choose to assign themes to individual reviewers.
>>>>>> The reviewer at that point will still need to go to the ticket and
>>>>>> mark it as "review" to let the theme authors know that it's being
>>>>>> actively reviewed.
>>>>>>
>>>>>> The goal is to streamline the process, make theme authors know what's
>>>>>> going on better, and to eliminate the manual portions here.
>>>>>>
>>>>>> Eventually, we hope to simplify the newer-version part as well, but
>>>>>> for now, the standard rule applies. If you pick a theme in the queue,
>>>>>> the first thing to do is to check if any newer versions of the theme
>>>>>> exist, and review the newer version while marking the old versions as
>>>>>> "closed-newer-version-uploaded". Please, again, do NOT mark a theme as
>>>>>> closed-newer-version-uploaded without doing a review of the latest
>>>>>> version. This is to maintain the queue ordering.
>>>>>>
>>>>>> Hope that helps. More changes to come. :)
>>>>>>
>>>>>> -Otto
>>>>>> _______________________________________________
>>>>>> 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 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130522/f4770894/attachment-0001.html>


More information about the theme-reviewers mailing list