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

Beschreibungstext von about spiegelt JSON Schema nicht wieder #219

Closed
Tracked by #113
kulla opened this issue Apr 21, 2023 · 3 comments · Fixed by #255
Closed
Tracked by #113

Beschreibungstext von about spiegelt JSON Schema nicht wieder #219

kulla opened this issue Apr 21, 2023 · 3 comments · Fixed by #255
Milestone

Comments

@kulla
Copy link
Contributor

kulla commented Apr 21, 2023

Mir ist aufgefallen, dass bei about (und auch bei anderen Eigenschaften, die aktuell nach Schema type === "Concept" haben), die Beschreibungstexte in der Spec nicht mit dem JSON Schema übereinstimmen. Im JSON Schema gibt es Angaben zu type, inScheme und prefLabel, die nicht im Beschreibungstext vorkommen.

@kulla
Copy link
Contributor Author

kulla commented Apr 21, 2023

Ein weiteres Beispiel hierfür ist learningResourceType

@acka47
Copy link
Member

acka47 commented Apr 24, 2023

Dieses Ticket ist verwandt mit #166 und berührt auch die Diskussion in #204.

@kulla kulla added this to the Version 1.0 milestone Apr 25, 2023
@acka47
Copy link
Member

acka47 commented May 8, 2023

Im JSON Schema gibt es Angaben zu type, inScheme und prefLabel, die nicht im Beschreibungstext vorkommen.

Meinetwegen können wir das rausnehmen. Die Informationen finden sich ja bei der verlinkten Ressource. M.E. ist es eine Best Practice, den Type und ein Label anzugeben (um z.B. für die Anzeige in einer Anwendung nicht parallel auch die verlinkte Ressource aufrufen zu müssen). Allerdings bin ich mir unsicher, ob wir das in der Spec fordern sollten,

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

Successfully merging a pull request may close this issue.

2 participants