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

103 orcid contributor affiliation selected randomly #128

Merged
merged 2 commits into from
Mar 4, 2024

Conversation

rekt-hard
Copy link
Collaborator

Description

What does this PR do?

Information for new contributors with ORCID identifier set will be fetched on DMP create/update. Affiliation will be selected to be current, with the earliest start date.

Since the information for email and the affiliations is scattered around the person attribute and the summary attribute, the whole ORCID record will be fetched. Required classes have been specified.

The ORCID test record resource is from the public ORCID API and the name has been modified.

Breaking changes

ORCID service will retrieve information from the /<orcid>/record endpoint.

Code review focus

Dependencies

Checks

  • Tested with Oracle/PostgreSQL
  • Export updated
  • Documentation added
  • Tests added
  • E2e tests created
  • Successfully ran e2e tests before merge

closes #103

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions

63.8% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@ValentinFutterer ValentinFutterer self-requested a review December 18, 2023 08:05
@rekt-hard rekt-hard force-pushed the 103-orcid-contributor-affiliation-selected-randomly branch 2 times, most recently from 96c0a7d to 5c3d324 Compare January 22, 2024 18:05
@rekt-hard rekt-hard force-pushed the 103-orcid-contributor-affiliation-selected-randomly branch from 5c3d324 to 66590bb Compare February 1, 2024 08:41
Copy link

sonarqubecloud bot commented Feb 1, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

60.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@rekt-hard rekt-hard force-pushed the 103-orcid-contributor-affiliation-selected-randomly branch from 66590bb to 304ffba Compare February 29, 2024 15:16
@ValentinFutterer ValentinFutterer self-requested a review March 4, 2024 09:35
@rekt-hard rekt-hard merged commit 8316c75 into next Mar 4, 2024
1 check failed
@rekt-hard rekt-hard deleted the 103-orcid-contributor-affiliation-selected-randomly branch April 2, 2024 15:53
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.

ORCID contributor affiliation selected randomly
2 participants