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

Make "type" array with mandatory default "LearningResource" #22

Closed
wants to merge 2 commits into from

Conversation

acka47
Copy link
Member

@acka47 acka47 commented Sep 24, 2020

Resolves #21

@acka47
Copy link
Member Author

acka47 commented Sep 24, 2020

@fsteeg @mirjan-hoffmann Mir fiel heute auf, dass das Schema Wenn ich das richtig sehe, wird type in OERSI gerade gar nicht verwendet. Das heißt wir müssten lediglich den default type LearningResource im ETL-Prozess ergänzen, um mit dieser Änderung konform zu gehen.

Es wird allerdings mit dem mergen eh noch ein wenig dauern, weil der SkoHub Editor damit nicht mehr wie vorgesehen funktioniert, siehe skohub-io/skohub-editor#54

@acka47
Copy link
Member Author

acka47 commented Oct 15, 2020

Note to self: Wenn wir die Anpassung am Schema machen, muss der SkoHub-Index nochmal gelöscht werden (siehe skohub-io/skohub-pubsub#40 (comment)).

@acka47
Copy link
Member Author

acka47 commented Nov 3, 2020

Da SkoHub Editor noch nicht mit dem Default-Wert klar kommt, mach ich das hier mal zu und mache einen PR für eine einfachere Variante auf.

@acka47 acka47 closed this Nov 3, 2020
@acka47 acka47 deleted the 21-LearningResource branch January 8, 2021 15:14
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 this pull request may close these issues.

Erlaube mehrere type-Werte
1 participant