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

learningResourceType - Array? #38

Closed
TobiasNx opened this issue Dec 9, 2020 · 4 comments · Fixed by #50
Closed

learningResourceType - Array? #38

TobiasNx opened this issue Dec 9, 2020 · 4 comments · Fixed by #50
Assignees

Comments

@TobiasNx
Copy link
Contributor

TobiasNx commented Dec 9, 2020

Aktuell ist learningResourceType ein Objekt, welches sowohl die id, als auch den Namen umfasst. Damit kann man aber mögliche Mischformen nicht berücksichtigen.

Mein Beispiel wäre, dass Videos auch Kurse oder Präsentationen sein können. @acka47 meinte dazu, dass es wenig Überschneidungen gebe. Video sei, wenn ein Bestandteil eines Kurses und Präsentation beziehe sich auf die Slides.

@acka47 und ich waren uns einig. dass wir es aber einmal diskutieren sollten.

@acka47
Copy link
Member

acka47 commented Jan 13, 2021

Im letzten Gruppentreffen haben wir beschlossen (siehe Notizen), hier einen Array zu verlangen. Das ist auch analog zur LOM-Spezifikation, siehe https://dini-ag-kim.github.io/hs-oer-lom-profil/latest/#das-element-learningresourcetype.

@acka47 acka47 assigned TobiasNx and unassigned acka47 Jan 13, 2021
@TobiasNx
Copy link
Contributor Author

Hi @acka47 , ich erhalte bei akj folgende Fehlermeldung: In schema.json kommt das Keyword auch nicht vor, wohl aber in allen Unterschemas.

bash test.sh
Testing version: draft
schema draft/schemas/schema.json is invalid
error: strict mode: unknown keyword: "_display"

Kannst du mir da helfen?

@acka47
Copy link
Member

acka47 commented Jan 14, 2021

Das ist eines der Probleme mit neueren ajv-Versionen. Du installierst am besten ajv-cli version 3.0:

$ npm install -g [email protected]

Siehe auch die .travis.yml.

@TobiasNx
Copy link
Contributor Author

Danke. Validiert jetzt, super!

@acka47 acka47 linked a pull request Jan 14, 2021 that will close this issue
@acka47 acka47 added this to AMB Jul 8, 2024
@acka47 acka47 moved this to Done in AMB Jul 8, 2024
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