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

[Anthony Whyte]

My own opinion is that in cases like what Bryan and Matt discuss below we should operate under the rules of lazy consensus after the proposal is posted on the sakai-dev list.  As I suggested in San Diego I would extend this approach to maintenance release work as well, letting those intimate with the release utilize lazy consensus rules throughout the release cycle including setting the release date.

Anth



On Aug 2, 2013, at 11:29 AM, Bryan Holladay wrote:

> I just set the default group property to true at r128112 for QA servers.
> 
> Just as a note, I didn't send to the TCC list or was expecting an official vote.  I just was looking for community voting and feedback.  The TCC is just so active that it usually turns into an unofficial TCC vote :)
> 
> 
> On Fri, Aug 2, 2013 at 11:20 AM, Matthew Jones <matthew at longsight.com> wrote:
> Perhaps this feature isn't worthy of a TCC vote? I think we should have some clarity on what the TCC would decide on leading up to the next release.
> 
> In 2011 prior to 2.9 prior to 2.9 we voted on many things similar to this: https://confluence.sakaiproject.org/display/TCC/2011+TCC+Voting+Summary. But does a new feature to an existing tool that passes QA need a specific vote?
> 
> There are two *major* new user facing additions so far (that I know of) for assignments. The group submission feature (which is also disabled by default SAK-22282 and currently has an open issue SAK-23697) and this one.
> 
> We should also turn on the group submission feature on the nightlies if it isn't already.
> 
> 
> 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 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"
> 
> 
> -- 
> 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"
> 
> 
> 
> _______________________________________________
> sakai2-tcc mailing list
> sakai2-tcc at collab.sakaiproject.org
> http://collab.sakaiproject.org/mailman/listinfo/sakai2-tcc

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/sakai2-tcc/attachments/20130802/99fa3279/attachment-0001.html