-
Notifications
You must be signed in to change notification settings - Fork 7
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
1806 manufacture #1896
1806 manufacture #1896
Conversation
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.
Yes.
d975589
to
77c9b96
Compare
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.
As commented
"manufacture" : [ { | ||
"startDate" : "1994", | ||
"type" : [ "Event" ], | ||
"location" : [ "EMI Records Ltd, p 1969" ] |
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 seems wrong. It should rather read "manufacturer" or similar and we should put the actual place of manufacture in this field.
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 is written in:
$a - Place of production, publication, distribution, manufacture (R)
this seems to be a cataloguing error.
77c9b96
to
8e3d3dc
Compare
"startDate" : "2010", | ||
"type" : [ "Event" ], | ||
"location" : [ "Marsberg" ], | ||
"manufacturedBy" : [ "Boxberger" ] |
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.
@acka47 This is a better example.
@dr0i you can merge this |
ahh, but first I have to update the test |
Will be deployed next Monday. |
The |
Resolves #1806
Add manufacture analog to publication.
Needs to be done/answered, @acka47 can you help:
sdo:Event
is good enough, since it is no PublishingEvent and Event is the "parent"(?) class.@blackwinter is this what you had in mind?