[theme-reviewers] New review workflow

[Srikanth Koneru]

No, name and email were both empty, added them now.


On Thu, May 23, 2013 at 12:29 AM, Chip Bennett <chip at chipbennett.net> wrote:

> 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
>>
>>
>
> _______________________________________________
> 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/20130523/9118ec39/attachment.html>