[samigo-team] Review rwequest: SAM-2088 GB registration

[Karen Tsao]

Hi David,

Your changes look good to me. I think Assignment is implemented in this
way. Am I correct?

As you mentioned this requires conversion script, are you going to provide
them as well?

Thanks,
Karen



On Wed, May 22, 2013 at 3:14 AM, David Horwitz <david.horwitz at uct.ac.za>wrote:

>  Hi All,
>
> I've checked a fix into a branch I would appreciate eyes on. It changes
> the way Samigo registers a assessment id with the gradebook from the format:
>
> [id] - the id of the published assessment
> to
>
> /samigo/[id]  - where [id] is the published assessment id
>
> The reasons for this are:
>
> 1) Performance: the external integration code in Samigo can identify
> references that are not part of its domain without going to  storage
> 2) Safety: it removes the risk that a similarly written integration would
> collide and give incorrect results in the gradebook
>
>
> note is will require a db conversion script.
>
> Feedback welcomed
>
> Regards
>
> David
>
>
> ------------------------------
> UNIVERSITY OF CAPE TOWN
>
> This e-mail is subject to the UCT ICT policies and e-mail disclaimer
> published on our website at
> http://www.uct.ac.za/about/policies/emaildisclaimer/ or obtainable from +27
> 21 650 9111. This e-mail is intended only for the person(s) to whom it is
> addressed. If the e-mail has reached you in error, please notify the
> author. If you are not the intended recipient of the e-mail you may not
> use, disclose, copy, redirect or print the content. If this e-mail is not
> related to the business of UCT it is sent by the sender in the sender's
> individual capacity.
>
> _______________________________________________
> samigo-team mailing list
> samigo-team at collab.sakaiproject.org
> http://collab.sakaiproject.org/mailman/listinfo/samigo-team
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/samigo-team/attachments/20130524/0293ce35/attachment.html