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
items: Add identifiers object list field #1205
items: Add identifiers object list field #1205
Conversation
from marshmallow import EXCLUDE, Schema, fields | ||
|
||
|
||
class IdentifiersSchema(Schema): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this schema already exists under document.py, it would be better to reuse it than create a duplicate
General: it looks we are missing the identifier schemes json update, to add a new identifier schema type ("call_number", which appears in the production script PR) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! do we need any upgrade recipe since a new field is added?
at least I think mapping update (check our internal docs how to do it)
we will need to have a plan for the deployment
@@ -16,6 +16,7 @@ | |||
set_changed_by, | |||
) | |||
from invenio_app_ils.records.loaders.schemas.price import PriceSchema | |||
from invenio_app_ils.documents.loaders.jsonschemas.document import IdentifierSchema |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Identifiers schema to be moved to records.loaders (like above)
❤️ Thank you for your contribution!
Description
Closes: CERNDocumentServer/cds-ils#834