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

Property hasInvolvement is required, clean-up around Involvement #359

Open
JuergenGrupp opened this issue Oct 8, 2024 · 0 comments
Open

Comments

@JuergenGrupp
Copy link
Collaborator

JuergenGrupp commented Oct 8, 2024

The class Involvement is referenced from EditorialObject via hasContributor or hasPublisher or hasCreator . While the latter two (hasPublisher, hasCreator) are relicts from the time before Involvement did exist and both pointed to Agent, the first (hasContributor) was introduced, when the class Involvement was still named Role.
Proposed changes:

  • For the current modelling, the property that links EditorialObject and ProductionJob to Involvement should be named hasInvolvement according to our naming conventions.

  • hasCreator should be kept as property of Annotation , but it should link to Agent , not to Involvement , as it currently does.

  • hasPublisher should be removed, as it is fully replaced by an Involvement classified as a publisher.

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

No branches or pull requests

1 participant