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

[publication] Allow same name lead investigators #9414

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

victori444
Copy link

@victori444 victori444 commented Oct 17, 2024

Brief summary of changes

Allows lead investigators with the same name but different emails to exist with distinct PublicationCollaboratorIDs in the Publications module

Testing instructions (if applicable)

  1. Either modify or create two publications: enter the same name for the lead investigator but enter a different email for both and check that both can save
  2. Verify that both exist with the correct emails in the publication_collaborator table under different IDs
  3. Modify one of the lead investigators' emails and verify that the other is not affected

CCNA Override

@victori444 victori444 added the Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant