WIPACrepo / keycloak-rest-services

Services surrounding KeyCloak, that use the REST API to read/update state
MIT License
1 stars 1 forks source link

[minor] Institution sync #34

Closed dsschult closed 2 years ago

dsschult commented 2 years ago

A script to import LDAP institution memberships into Keycloak.

Some other changes:

dsschult commented 2 years ago

Going to try doing something with the author list before finishing this.

ric-evans commented 2 years ago

What's your plan for retired/inactive institutions?

briedel commented 2 years ago

Just a general note... It might be better to have "region": _US_CAN be "region": _NA_CAN

ric-evans commented 2 years ago

Just a general note... It might be better to have "region": _US_CAN be "region": _NA_CAN

Is that North America?

briedel commented 2 years ago

Is that North America?

Yes, or whatever else you want to put in there for North America

ric-evans commented 2 years ago

Is that North America?

Yes, or whatever else you want to put in there for North America

Right now this gets expanded to "US and Canada". I'm not sure where this distinction came from (or is used) and why we don't use "North America".

dsschult commented 2 years ago

Is that North America?

Yes, or whatever else you want to put in there for North America

Right now it appears as:

_US_CAN = "US and Canada"

So we also should change that string. Maybe just "North America" is fine?

@ric-evans is that important for MOUs?

ric-evans commented 2 years ago

@ric-evans is that important for MOUs?

No, it's not. The MOUs only care about the "is_US" field which decides whether NSF funds are applicable.

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging a02f5688534a40d63d896919aa7cacca4f082bad into d2d0d282f05edbbbe9ec4ad947004f047d3e4231 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 27b3151bb03dc36001833e8ff8c8968473f76d1b into d2d0d282f05edbbbe9ec4ad947004f047d3e4231 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging e306aad9bf82d36272ea161ceade934575c38ebe into d2d0d282f05edbbbe9ec4ad947004f047d3e4231 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging d3a27cc2e4dfb2f922373a2496cc6bd552b5a9e3 into d2d0d282f05edbbbe9ec4ad947004f047d3e4231 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging e7ce51d2fdb07391e0e20ef9a898526a9a429191 into bead04908a6e8d7cd080fadbfc3bec7017fcede1 - view on LGTM.com

new alerts: