DukeLearningInnovation / kits

https://dukelearninginnovation.github.io/kits
GNU General Public License v3.0
37 stars 5 forks source link

Changing student role to TA role in Kits - propogate to all apps #338

Open profmikegreene opened 5 years ago

profmikegreene commented 5 years ago

Description

bumping an official student up to TA isn't coming over to Sakai, non-official assistants are mapping correctly. kit id 254 student andre.agassi@duke.edu

12/13/18 update: I'm not sure any after the fact permissions are currently coming over after the original sync. I did some testing in ARABIC 102 - 04D Sp19 on test this afternoon and Kits didn't add me to the Sakai site either.

This will also be the situation with Sites @ Duke and any other app with a separate user permission management. Because Kits allows an individual to have only one role within a course community, the deployment of Kits in production may change the source system of user permissions to Kits, requiring updates to roles across the apps.

ewendland23 commented 5 years ago

Michael, i started looking at this issue on Monday.

1) Old Toolkits never lets you change the Sakai role of people in the SISS groups. If it sees they are students or instructors it just does not try. This is what you are seeing now.

2) I played around with removing my protections, but I can't figure out how to get Sakai to update the role. I might need some help from a Sakai expert? Or is it even possible from the API? From this: https://dukedev.longsight.com/direct/membership/describe it looks like I can, but thus far i haven't combined the magic correctly.

profmikegreene commented 5 years ago

confirmed this is still an issue on kits/sakai test