-
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
release/v0.12.0 #61
release/v0.12.0 #61
Conversation
Add url()
Update figshare_group()
Add verbose option in get_current_groups
Fix calls to mo.update_dataframe when root case
log revert_command if membership is empty append quota to revert_command (including root)
- Change 'org_code' section to 'google' - Update get_org_code_numbers script for section name change
PEP8 formatting #6
Add docstrings #6
Adjust log messages for bool keys
add_grouper_groups: Handle quota flag when set
Use f-strings for log filenames
Add log_stdout function
import log_stdout
import log_stdout
Update script_run, user_update, get_org_code_numbers scripts
Update user_update to pass log
Update user_update script
Update script_run
Implement in all scripts
Implement in all scripts
user_update: Add log keyword option for GrouperAPI
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 complete.
Will need to test each script to confirm complete log messages
- Update requirements for pandas, tabulate - Implement in script_run - script_run: Strip out log info about org codes, org names
#62 - Optional mo keyword for ManualOverride object if needed - Implement changes in user_update script
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.
Final set of changes looks good for minor refactoring and improvements.
A number of tests were conducted and those passs
Fixes #42
Fixes #52
Fixes #57
Fixes #58
Fixes #60
Fixes #62
Fixes #63
Fixes #64
The following tests are needed since there are a number of features that overlap:
script_run
:pandas
DataFrame for each theme/portal (see figshare_patrons.2020-09-30.log)ManualOverride
file logging is available (optional) (see figshare_patrons.2020-09-30.log)GrouperQuery
file logging is available (optional) N/Auser_update
:grouper_delta_user
file logging is available (there was a minor code mistake - log written to stdout, fixed in e13b6b4)ManualOverride
file logging is available (optional) (see user_update.2020-09-30.log)LDAPConnection
file logging is available (optional) (see user_update.2020-09-30.log)GrouperAPI
file logging is available (optional) N/A, some improvements for KeyError handling (minor)add_grouper_groups
:GrouperAPI
file logging is available (optional) N/A, some improvements for KeyError handling (minor)get_org_code_numbers
: (this script takes some time to run, verification in code)LDAPConnection
file logging is available (optional)