FAQ | This is a LIVE service | Changelog

Commit 1c59156e authored by Robin Goodall's avatar Robin Goodall 💬
Browse files

Updates/fixes from review

parent bba8dd16
Pipeline #192750 passed with stages
in 3 minutes and 18 seconds
......@@ -3,8 +3,8 @@
Tool to query Lookup for CHRIS and CamSIS institutional membership.
> This tool is currently only has the one operation `student-inst-members` that is able to compare
> CamSIS student affiliations to Lookup group membership and update as appropriate. It is not able
> to create Lookup groups.
> CamSIS student affiliations to Lookup group membership and update as appropriate. If the
> appropriate lookup group doesn't exist then it is created first.
>
> Additionally, an operation to do the same for CHRIS institution membership is yet to be
> implemented.
......
......@@ -81,7 +81,7 @@ def compare_with_lookup_groups(
missing_groups.add(group)
# Will want to add everyone after creating the group
group_changes[group] = {'add': students, 'remove': set()}
LOG.info('Group "%s" needs creating')
LOG.info('Group "%s" needs creating with %s students', group, len(students))
continue
LOG.info('Group "%s" should have %s student(s):', group, len(students))
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment