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

Identify DOI in RERO DOC field 775 $o #696

Merged
merged 1 commit into from
Nov 10, 2021
Merged

Conversation

mmo
Copy link
Collaborator

@mmo mmo commented Nov 5, 2021

Co-Authored-by: Miguel Moreira [email protected]

Why are you opening this PR?

  • Which task/US does it implement?
  • Which issue does it fix?

How to test?

  • What command should I have to run to test your PR?
  • What should I test through the UI?

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

* When MARC field 775 $o contains a DOI in RERO DOC, its value is
  imported in the `identifiedBy` field instead of `otherEdition`.
* Closes rero#659.
Co-Authored-by: Miguel Moreira <[email protected]>
@mmo mmo force-pushed the mmo-identify-doi-659 branch from d62832b to d2789b6 Compare November 5, 2021 13:13
@mmo mmo marked this pull request as draft November 5, 2021 13:16
@mmo mmo marked this pull request as ready for review November 5, 2021 14:26
@jma jma merged commit 16d5356 into rero:staging Nov 10, 2021
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.

Identify all DOIs imported from RERO DOC
2 participants