Import invitations from a CSV#5108
Merged
ThiefMaster merged 2 commits intoindico:masterfrom Oct 26, 2021
Merged
Conversation
tomasr8
force-pushed
the
invitation-csv-import
branch
2 times, most recently
from
f59c1f5 to
871699b
Compare
ThiefMaster
reviewed
Oct 7, 2021
indico/modules/events/registration/templates/management/regform_invitations.html
Outdated
Show resolved
Hide resolved
indico/modules/events/registration/controllers/management/invitations.py
Outdated
Show resolved
Hide resolved
indico/modules/events/registration/controllers/management/invitations.py
Outdated
Show resolved
Hide resolved
tomasr8
force-pushed
the
invitation-csv-import
branch
4 times, most recently
from
6737bda to
a04ac6a
Compare
ThiefMaster
reviewed
Oct 25, 2021
indico/modules/events/registration/controllers/management/invitations.py
Outdated
Show resolved
Hide resolved
Member
|
Also, |
tomasr8
force-pushed
the
invitation-csv-import
branch
2 times, most recently
from
0c387c8 to
782dc19
Compare
tomasr8
force-pushed
the
invitation-csv-import
branch
from
782dc19 to
06c5ca3
Compare
ThiefMaster
reviewed
Oct 26, 2021
indico/modules/events/registration/controllers/management/invitations.py
Outdated
Show resolved
Hide resolved
ThiefMaster
reviewed
Oct 26, 2021
tomasr8
force-pushed
the
invitation-csv-import
branch
from
06c5ca3 to
48ba476
Compare
ThiefMaster
force-pushed
the
invitation-csv-import
branch
from
48ba476 to
8394534
Compare
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Closes #3673
One can now import invitations form a CSV similar to importing registrations.
Includes an option to skip existing invitations.
As for the implementation details, since the parsing is similar to registrations, I abstracted
import_registrations_from_csvintoimport_users_from_csvwhich does parsing and basic validationand is used by both
import_registrations_from_csvandimport_invitations_from_csv.