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

The default synonym property (skos/core#altLabel) does not handle multiple values #97

Open
mdorf opened this issue Nov 12, 2019 · 3 comments

Comments

@mdorf mdorf added the bug label Nov 12, 2019
@mdorf mdorf self-assigned this Nov 12, 2019
@jonquet
Copy link

jonquet commented Nov 12, 2019

I believe this has been fixed within AgroPortal branch. As illustrated by the call:
http://data.agroportal.lirmm.fr/ontologies/NCBITAXON/classes/http%3A%2F%2Fpurl.bioontology.org%2Fontology%2FNCBITAXON%2F2157?display=all

@vemonet Do you remember where this would be in the code ?

@graybeal
Copy link

@mdorf What would happen if we plugged #altLabel into the synonym properties list for these other ontologies?

@jonquet
Copy link

jonquet commented Apr 23, 2022

@syphax-bouazzouni do you confirmed this is fixed.

galviset referenced this issue in EarthPortal/ontologies_linked_data Mar 14, 2024
* fix  json serializer nil exception (#79)

if hash was missing @context key the json serializer throwed a nil exception

* Merge pull request #72 from ontoportal-lirmm/feature/migrate-attrbiutes-metadata-to-scheme-file

Feature: Update ontology submission model 2023 - Part 1  -  Update attributes validators and metadata

* Merge pull request #82 from ontoportal-lirmm/feature/embed-meterics-for-submissions

Feature: embed metrics for submission model

* Merge pull request #86 from ontoportal-lirmm/fix/read_restricted_based_on-nil

Fix: call to read_restricted?  only if instance_to_base_on not nil

* Merge pull request #90 from ontoportal-lirmm/feature/add-persons-and-organization-models

Feature: Update ontology submission model 2023 - Part 2 - Add person & organization models

* in tests no more use the reload argument in the exist? method

---------

Co-authored-by: Raimi Solorzano Niederhausen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants