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

holdings: fix is_serial definition #1648

Merged
merged 1 commit into from
Jan 28, 2021
Merged

Conversation

BadrAly
Copy link

@BadrAly BadrAly commented Jan 27, 2021

Previously, a holdings is considered of type serial
if it is attached to document of type journal. With
this commit, if the holdings has the holdings_type =
serial, it is considered a serial holdings.

  • Adapts units testing
  • Shortens the title of the holdings fields, template and patterns
  • Makes the fields EnumerationAndChronolgy as textarea

Co-Authored-by: Aly Badr [email protected]

@BadrAly BadrAly added this to the v1.0.0 milestone Jan 27, 2021
@BadrAly BadrAly self-assigned this Jan 27, 2021
Copy link
Contributor

@jma jma left a comment

Choose a reason for hiding this comment

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

In the commit message each item list should end with a ".".

@lauren-d
Copy link
Contributor

Be careful, you have 2 commits

Previously, a holdings is considered of type serial
if it is attached to document of type journal. With
this commit, if the holdings has the holdings_type =
serial, it is considered a serial holdings.

* Adapts units testing.
* Shortens the title of the holdings fields, template and patterns.
* Makes the fields EnumerationAndChronolgy as textarea.

Co-Authored-by: Aly Badr <[email protected]>
@BadrAly BadrAly merged commit 3345583 into rero:dev Jan 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants