-
Notifications
You must be signed in to change notification settings - Fork 2
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
feature/87_grouper_refactor #117
Conversation
- Implement migration in script_run - Change input parameters for Delta to use dict instead of GrouperQuery instance - Simplify grouper auth with attribute
- Refactored into grouper module
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look and --sync
or --add
testings was completed with scripts impacted by this refactor.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved with a minor comment. I'll hold off on merging until you decide what to do.
See #87
The following tests will need to occur manually to check that it is working with
--sync
:script_run
user_update
add_grouper_groups