Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove passed freshmen from the 10weeks ldap group after passing 10 weeks #209

Open
devinmatte opened this issue Jan 22, 2019 · 2 comments

Comments

@devinmatte
Copy link
Member

There is now a 10weeks ldap group, to help show and determine who is currently in the 10 weeks process to services. Members should be removed from this upon passing a 10 weeks but remain in the intromembers group until they pass a spring evaluations

@mxmeinhold
Copy link
Contributor

It would be nice to have conditional handle all the group updates, to prevent an RTP from having to do it, but with intro evals we run into an issue of information being exposed by house services before evals can distribute results. My thoughts on design for this (and for disabling failed accounts):

  • When a result is selected in the slideshow, insert a row into the db (some new queue table or something) with the ldap changes to be effected, without actually making changes in ldap. Perhaps these should be abstract (e.g. 'fail-10weeks' rather than 'remove from 10weeks group') so that behavior can be changed in code without needing to update db fields.
  • Have an evals only or similarly restricted page that displays these queued changes and allows evals to dispatch them one by one, so evals can wait to hit the button until after distributing results.

Also, I have some extra ldap info that should be updated after intro evals: In the event of a failed result, a user should be removed from every group other than current_student (e.g. [10weeks, intromembers, onfloor, active, member]) and the account should be disabled. I don't remember if disabling can be done via csh_ldap, or if we'll need to setup something else to talk to ipa.

@mxmeinhold
Copy link
Contributor

This is related to #136

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants