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

holding: fix holding display problem. #378

Merged
merged 1 commit into from
Oct 7, 2020

Conversation

zannkukai
Copy link
Contributor

If a document is a serial or electronic, all linked holdings was force
to be displayed as serial/electronic. This PR fix this problem ; despite
of the document metadata, the holding is display depending of the
holding_type metadata from the holding.

Closes rero/rero-ils#1252

Co-Authored-by: Renaud Michotte renaud.michotte@gmail.com

Why are you opening this PR?

rero/rero-ils#1252

How to test?

  • display "serial/electronic" documents detailed view and check holdings display.

Code review check list

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

@zannkukai zannkukai self-assigned this Oct 1, 2020
If a document is a serial or electronic, all linked holdings was force
to be displayed as serial/electronic. This PR fix this problem ; despite
of the document metadata, the holding is display depending of the
holding_type metadata from the holding.

Closes rero/rero-ils#1252

Co-Authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai marked this pull request as ready for review October 5, 2020 08:33
@zannkukai zannkukai merged commit 949846c into rero:dev Oct 7, 2020
@zannkukai zannkukai deleted the zan-fix-pilot-holdings branch October 16, 2020 11:27
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.

Holdings for journals have various problems
3 participants