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

knora-admin ontology: project description cardinality #875

Closed
lrosenth opened this issue Jun 1, 2018 · 4 comments · Fixed by #886
Closed

knora-admin ontology: project description cardinality #875

lrosenth opened this issue Jun 1, 2018 · 4 comments · Fixed by #886
Assignees
Milestone

Comments

@lrosenth
Copy link
Collaborator

lrosenth commented Jun 1, 2018

@subotic @benjamingeer
The old salsah now supports multiple project descriptions (for different languages). Therefore the
cardinality in knora-admin.ttl for the project descriptions should be changed to either

                          [ rdf:type owl:Restriction ;
                            owl:onProperty :projectDescription ;
                            owl:minCardinality "0"^^xsd:nonNegativeInteger
                          ] ,

or

                          [ rdf:type owl:Restriction ;
                            owl:onProperty :projectDescription ;
                            owl:minCardinality "1"^^xsd:nonNegativeInteger
                          ] ,

I would prefer the latter variant where we force to have at least one project description.
Who will change this – are there any objections?

@benjamingeer
Copy link

Seems fine to me. In that case I guess it should also have a language tag.

@subotic
Copy link
Collaborator

subotic commented Jun 1, 2018

In that case I guess it should also have a language tag.

It already does :-) It is a Seq[StringLiteralV2]. We would only need to change knora-admin.ttl and check that the payload for creating a project must at least contain one project description.

@lrosenth
Copy link
Collaborator Author

lrosenth commented Jun 1, 2018 via email

@subotic
Copy link
Collaborator

subotic commented Jun 12, 2018

work-in-progress in #886

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants