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

notes: allow notes of short size #1476

Merged
merged 1 commit into from
Nov 23, 2020
Merged

Conversation

BadrAly
Copy link

@BadrAly BadrAly commented Nov 20, 2020

Sets the note minimum length of holdings, patron and item to 1 instead
of 3 characters, to allow libraries to use notes as codes for some
internal uses cases.

Co-Authored-by: Aly Badr aly.badr@rero.ch

Why are you opening this PR?

https://tree.taiga.io/project/rero21-reroils/task/1880?kanban-status=1224895

Dependencies

My PR depends on the following rero-ils-ui's PR(s):

  • rero/rero-ils-ui#

How to test?

You can not add holdings/patron/item notes of size 1

Code review check list

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

@BadrAly BadrAly self-assigned this Nov 20, 2020
@BadrAly BadrAly added the f: editor Concerns editor based on JSON schema AND custom editor label Nov 20, 2020
@iGormilhit iGormilhit added f: data About data model, importation, transformation, exportation of data, specific for bibliographic data and removed f: editor Concerns editor based on JSON schema AND custom editor labels Nov 23, 2020
Copy link

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

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

Commit message body:

Sets the note minimum length of holdings, patron and item to 1 instead
of 3 characters, to allow libraries to use notes as codes for some
internal uses cases.

@iGormilhit iGormilhit added this to the v0.14.1 milestone Nov 23, 2020
Sets the note minimum length of holdings, patron and item to 1 instead
of 3 characters, to allow libraries to use notes as codes for some
internal uses cases.

Co-Authored-by: Aly Badr <aly.badr@rero.ch>
Copy link

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

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

Commit message approved.

@BadrAly BadrAly merged commit 36319da into rero:dev Nov 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f: data About data model, importation, transformation, exportation of data, specific for bibliographic data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants