[VYMCA-115] Admin accounts for CSV import #264
Merged
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.
Related Issue/Ticket:
Some Y's are having issues with giving admin access to the staff members that exist in the CSV file.
Steps to reproduce:
- Run import
- Edit any imported user and assign a "virtual editor" role
- Run import one more time
- Check that user without a "virtual editor" role
Solution:
Add settings with the user's emails list that should be skipped during migration.
Steps to test:
/admin/modules
/admin/openy/virtual-ymca/gc-auth-settings
/admin/openy/virtual-ymca/gc-auth-settings/provider/custom
test.csv
file with content:/admin/openy/openy-gc-auth/settings/provider/custom/upload-csv
/admin/structure/migrate/manage/gc_auth/migrations/gc_auth_custom_users/execute
/admin/people
/admin/openy/virtual-ymca/gc-auth-settings/provider/custom
/admin/structure/migrate/manage/gc_auth/migrations/gc_auth_custom_users/execute
/admin/people