Skip to content
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

Have guid-->email lookup strip underscore+suffix for multiple submissions from same user #98

Closed
j-wags opened this issue Nov 22, 2016 · 2 comments
Assignees
Milestone

Comments

@j-wags
Copy link
Contributor

j-wags commented Nov 22, 2016

No description provided.

@coleslaw481
Copy link
Contributor

What is going on here is the
participant_list.csv

will have an entry like so in participant database:

name,d3rusername,guid,email
bob smith,bsmith,12345,bob@bob.com

In the example above if bob wants to submit twice for a given celpp week we plan to just create an additional shared folder with the following name: 12345_# where # is a number starting at say 2 so 12345_2 12345_3...

The above works fine for processing, but when calling get_participant_by_guid() from ParticipantDatabase with the the guid 12345_2 no entry will be found. The method get_participant_by_guid() needs to become smarter. It should first try an exact match of the guid and if that fails see if _# exists and if so strip it off and then look for a guid.

If found return that participant, but update a new getter/setter in the Participant object that is named submission number. By default this value will be None.

Once changes above are made code downstream needs to be examined to verify no bugs arise from this update.

@coleslaw481 coleslaw481 added this to the 1.6.3 milestone Nov 23, 2016
coleslaw481 added a commit that referenced this issue Nov 29, 2016
…is stripped off end and search is performed again
@coleslaw481
Copy link
Contributor

Fixed in 363a56d..018e659 chrisdev -> chrisdev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants