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

Fix: Disable multiple identifiers for agents #730

Merged

Conversation

Bilelkihal
Copy link
Collaborator

Fix the issues 2 and 3 here:
agroportal/project-management#598 (comment)

Done in this PR:

@Bilelkihal Bilelkihal self-assigned this Aug 6, 2024
@syphax-bouazzouni syphax-bouazzouni self-requested a review August 8, 2024 06:13
Copy link
Collaborator

@syphax-bouazzouni syphax-bouazzouni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test fixed

@syphax-bouazzouni syphax-bouazzouni merged commit 3da41db into development Aug 8, 2024
4 checks passed
syphax-bouazzouni added a commit that referenced this pull request Aug 8, 2024
* fix: identifier for organizations is ror and for persons is orcid

* disable multiple identifiers for agents in agents form

* remove the restriction for certain submission fields to create only organization type or person type agents

* fix agent tests after changing agent identifier to not be a list

---------

Co-authored-by: Syphax <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants