Assume program is Alumni if UWLDAP is missing data #84

Merged
merenber merged 1 commits from uwldap-assume-alumni into master 3 months ago
Owner

At some point, I think that UWLDAP had 'ou=Alumni' for alumni, but that is no longer the case. Now, alumni simply don't have an 'ou' attribute at all in UWLDAP. As a result, the updateprograms ceo command currently does not update the CSC LDAP 'program' attribute for people who recently graduated.

This PR sets 'program=Alumni' for members who either do not have an 'ou' attribute in UWLDAP, or who do not have a UWLDAP entry at all.

At some point, I think that UWLDAP had 'ou=Alumni' for alumni, but that is no longer the case. Now, alumni simply don't have an 'ou' attribute at all in UWLDAP. As a result, the `updateprograms` ceo command currently does not update the CSC LDAP 'program' attribute for people who recently graduated. This PR sets 'program=Alumni' for members who either do not have an 'ou' attribute in UWLDAP, or who do not have a UWLDAP entry at all.
merenber added 1 commit 3 months ago
950e794bb7 assume alumni if UWLDAP is missing data
merenber requested review from r389li 3 months ago
r389li approved these changes 3 months ago
r389li left a comment
Owner

LGTM, thanks merenber

LGTM, thanks merenber
merenber merged commit 1e452d10ce into master 3 months ago
merenber deleted branch uwldap-assume-alumni 3 months ago

Reviewers

r389li approved these changes 3 months ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 1e452d10ce.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: public/pyceo#84
Loading…
There is no content yet.