[sakai-pmc] Help on SAM-1943

Mark J. Norton markjnorton at earthlink.net
Mon Feb 10 07:27:17 PST 2014


I see.  Well, I leave it in the capable hands of the PMC.  :)

- Mark

On 2/10/2014 10:20 AM, Sam Ottenhoff wrote:
> >   What does checkMembershipForGroupRelease() and 
> isUserInAuthorizedGroup() have to do with adding a new question type?
>
> Absolutely nothing.  It's a gigantic patch that includes hundreds of 
> comment changes and lots of code cleanup.  In reality, it looks like 
> it should probably be split up into many JIRAs addressing all of the 
> potential changes incorporated by the mega patch.  I've left my 
> comments in the code review.  Jacques replied that he can't make the 
> patch smaller:
>
> https://crucible.sakaiproject.org/cru/SAKTRUNK-37
>
>



More information about the sakai-pmc mailing list