[sakai2-tcc] [Building Sakai] New Feature: Assignment Peer Review SAK-23812

[Bryan Holladay]

I've set the property to "true" for all QA servers @r128109


On Fri, Aug 2, 2013 at 11:02 AM, Matthew Jones <matthew at longsight.com>wrote:

> We talked internally, there really isn't a clear cut process, but
> generally I think all new features should be property protected even in
> trunk, at least the UI piece.
>
> This should be disabled by default, then we can enable the property on the
> nightly servers so it's visible and can be tested. If all testing goes well
> then TCC can vote prior to release to turn these features on by default.
> (or possibly even remove the property)
>
> -Matthew
>
>
> On Fri, Aug 2, 2013 at 10:54 AM, Bryan Holladay <holladay at longsight.com>wrote:
>
>> Thanks for the feedback.  I've committed this to trunk.
>>
>> One last question:
>>
>> Should this feature only be available through a Sakai.Property, which is
>> current set to False by default?  Should I set it to "True" by default or
>> just remove it?
>>
>> -Bryan
>>
>>
>> On Wed, Jul 31, 2013 at 3:05 PM, Alex Balleste <
>> alexandre.balleste at udl.cat> wrote:
>>
>>>  It looks great. We will try it.
>>>
>>> Thanks for that new feature.
>>>
>>> Alex
>>>
>>>
>>>
>>> On 07/30/2013 09:04 PM, Bryan Holladay wrote:
>>>
>>> I have completed a new feature sponsored by NYU to allow Peer Reviews in
>>> the Assignments tool and I'd like to get this code into trunk.  I've talked
>>> with Zhen and she will be out on vacation for the next two weeks so she
>>> said to go to the list.  I've linked the documentation [1] and screen casts
>>> [2][3] as well as the Jira [4].  It's currently setup as a property feature
>>> (off by default), but I'd prefer no property mainly because properties tend
>>> to stay off and not get tested properly (not a deal breaker for me).  A
>>> patch for 2.10 (4.0? 10.0?) and 2.9.x is attached to the jira.  Derek
>>> Ramsey has spent a lot of time QA'ing this feature.
>>>
>>>  I'm looking for a vote at a minimum and if someone wants to review the
>>> patch, that'd be swell.
>>>
>>>  +1 from me :)
>>>
>>>  Thanks,
>>> Bryan
>>>
>>>
>>>  [1]
>>> https://docs.google.com/document/d/1KMoIOGjT9-1d1jc8LTIx6-75BQb95qjYJymhcKFXkSY/pub
>>> [2] Video Part 1: http://screencast.com/t/SX8zjMOky
>>> [3] Video Part 2: http://screencast.com/t/tsC3bHtn
>>>  [4] https://jira.sakaiproject.org/browse/SAK-23812
>>>
>>>
>>> _______________________________________________
>>> sakai-dev mailing listsakai-dev at collab.sakaiproject.orghttp://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>>>
>>> TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe"
>>>
>>>
>>>
>>> --
>>> Alexandre Ballesté Crevillén  alexandre.balleste at udl.cat
>>> =====================
>>> University of Lleida
>>> Information and Communication Systems Service
>>> Analyst / Programmer
>>> Twitter: @sinmsinm
>>>
>>> Tlf: +34 973 702148
>>> Fax: +34 973 702130
>>> =====================
>>>
>>>
>>> _______________________________________________
>>> sakai-dev mailing list
>>> sakai-dev at collab.sakaiproject.org
>>> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>>>
>>> TO UNSUBSCRIBE: send email to
>>> sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of
>>> "unsubscribe"
>>>
>>
>>
>> _______________________________________________
>> sakai-dev mailing list
>> sakai-dev at collab.sakaiproject.org
>> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>>
>> TO UNSUBSCRIBE: send email to
>> sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of
>> "unsubscribe"
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/sakai2-tcc/attachments/20130802/cb810c65/attachment.html