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

feat: new organizationTags json #98

Merged
merged 3 commits into from
Nov 23, 2023

Conversation

ThorstenDiekhof
Copy link
Contributor

@ThorstenDiekhof ThorstenDiekhof commented Nov 23, 2023

eine json mit generierten Kategorien inklusive Übersetzungen.

Copy link
Collaborator

@herrherrmann herrherrmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich habe jetzt mal nur die deutschen und englischen Varianten gecheckt und noch ein paar Kleinigkeiten gefunden, die wir vor dem Merge ausbessern sollten.

"identifier": "organization.category.IndependentArtist",
"title": {
"de": "Freie Künstler:in / Gruppe / Kollektiv",
"en": "Free Artist / Group / Collective",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich glaube, hier wäre "Independent" statt "Free" passender (wie auch im identifier und bei den anderen Übersetzungen)?

Copy link
Contributor Author

@ThorstenDiekhof ThorstenDiekhof Nov 23, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unbedingt. Ich habe die Übersetzungen generieren lassen. Da finden sich sicherlich auch einige Fehler in den Sprachen, die ich nicht beherrsche. Wäre aber sicher wichtig, dass wir im Englischen korrekt sind. Danke dafür.

"type": "type.Tag",
"identifier": "organization.category.Theater.PuppetFigureObjectTheater",
"title": {
"de": "Puppen- / Figuren-/ Objekttheater",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier sind die Leerzeichen zwischen den Slashes nicht konsistent. 👮

Suggested change
"de": "Puppen- / Figuren-/ Objekttheater",
"de": "Puppen- / Figuren- / Objekttheater",

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super. Ich habe die Texte 1 zu 1 von der alten Webseite übernommen.

"type": "type.Tag",
"identifier": "organization.category.GalleryExhibitionSpaceInitiative",
"title": {
"de": "Galerie / Ausstellungsstätte / Initative für Bildende Kunst",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"de": "Galerie / Ausstellungsstätte / Initative für Bildende Kunst",
"de": "Galerie / Ausstellungsstätte / Initiative für Bildende Kunst",

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super. Ich habe die Texte 1 zu 1 von der alten Webseite übernommen. Gut, wenn wir da Fehler verbessern.

"identifier": "organization.category.LibraryArchive.SpecializedLibrary",
"title": {
"de": "Spezial- / Fachbibliothek",
"en": "Specialized / Fach Library",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Fach Library" ist hier wahrscheinlich nicht korrekt, oder? dict.cc sagt, das könnte entweder "special library" oder "technical library" sein.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ich denke hier wurde nicht special library genommen, um sich nicht zu wiederholen.
Ich denke Specialized Library reicht hier, oder?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ja, ich stimme zu – Specialized Library reicht hier!

@ThorstenDiekhof
Copy link
Contributor Author

Änderungen eingebaut.

Copy link
Collaborator

@herrherrmann herrherrmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super, ab damit!

@ThorstenDiekhof ThorstenDiekhof merged commit b5c6e29 into main Nov 23, 2023
4 checks passed
@ThorstenDiekhof ThorstenDiekhof deleted the feat/seed-for-organization-categroy-tags branch November 23, 2023 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants