Improvements to volunteer import script #219
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
struan
force-pushed
the
207-ceuk-volunteer-import-improvements
branch
3 times, most recently
from
4308d05 to
1b61f4f
Compare
This adds the following features: * a dry run mode to allow seeing what the end result will be without updating the database * a debug more which has diagnotstic information about how many assignments are being made to each user * an option to read assignment counts from the import file * an option to read preferred assignment council types (country or LBO only at the moment) from the import file It also will not assign Audit users to councils they marked as part of the First Mark. add dry_run option to volunteer import script Update the script to run all changes inside a transaction and if the dry_run option is passed then rollback all changes. This lets you see any issues with running out of assignments without having made any changes to the database. enable reading assignment counts from the import data allow volunteer import data to have council restrictions so you can limit people to e.g. Welsh councils. allow superusers and ceuk staff to have existing assignments Mostly to account for them having multuple rows in the same spreadsheet for non First Mark assignments do not assign to council already assigned This is mostly for the Audit stage where users should not be auditing their own work use response type in volunteer import where required add more bad section names to section name map add debug option so can output more verbose info change restricted councils to be councils to select from first
struan
force-pushed
the
207-ceuk-volunteer-import-improvements
branch
from
1b61f4f to
57a39f4
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This updates the volunteer import script to make it yet more flexible.
Adds the following:
--dry_runoption which runs everything inside a transaction and then rolls it back so you can get the full report of what it would do without changing the DB. Mostly to allow checking if it's going to assign everything required--council_restrictionsoption which indicates there is a column in the import data with details of council restrictions for some volunteers--section_count_in_datawhich indicates that the number of councils to assign to volunteers is contained in a column in the import data.Fixes #207