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.

Sign up for GitHub

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

Jump to bottom

Personen-Attribut titleOfNobility fehlt und muss berücksichtigt werden #118

Closed
xwolfde opened this issue Dec 13, 2024 · 2 comments
Closed

Personen-Attribut titleOfNobility fehlt und muss berücksichtigt werden #118

xwolfde opened this issue Dec 13, 2024 · 2 comments
Assignees
@xwolfde @MManthey
Labels
Abnahme ok Passt für die Milestoneversion bug Something isn't working Semantik Issues zu Semantik und strukturierte Daten
Milestone
V2.1 - Finetuning...

Comments

@xwolfde
Copy link
Member

xwolfde commented Dec 13, 2024 •
edited
Loading

Der Adelstitel bzw. Zwischentitel einer Person wurde von der API in "titleOfNobility" übergeben.

Wir müssen diesen Titel bei der Generierung des displayNames noch berücksichtigen und zwischen Vornamen und Nachnamen einfügen.

Testcase:

Daniel de West

Die API liefert bei ihm korrekt:
"titleOfNobility": "de"
@xwolfde xwolfde added the Semantik Issues zu Semantik und strukturierte Daten label Dec 13, 2024
@xwolfde xwolfde added this to the V2.2 - Finetuning II milestone Dec 13, 2024
@xwolfde xwolfde added the bug Something isn't working label Dec 13, 2024
@xwolfde
Copy link
Member Author

xwolfde commented Dec 13, 2024

Muss das doch als Bug angeben für euch:

Ihr hattet das schon berücksichtigt. In der Personen-Darstellung im Backend gibt es das Feld dazu schon:
grafik

Allerdings ist es bei Daniel leer, obwohl über die API auch kam:
grafik

MManthey added a commit that referenced this issue Dec 16, 2024
@MManthey
#118 - title of nobility
1b3dead
@MManthey MManthey added the Abnahmetest Dieser Issue wurde vom Entwickler als fertig gesehen und kann getestet/abgenommen werden label Dec 16, 2024
@xwolfde
Copy link
Member Author

xwolfde commented Dec 17, 2024 •
edited
Loading

Jupp, passt.

Muss noch bei der Erstellung des Displaynames ergänzt werden, aber das kann ich im Finetuning machen.

Ref #120

@xwolfde xwolfde added Abnahme ok Passt für die Milestoneversion and removed Abnahmetest Dieser Issue wurde vom Entwickler als fertig gesehen und kann getestet/abgenommen werden labels Dec 17, 2024
@xwolfde xwolfde closed this as completed Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xwolfde xwolfde

@MManthey MManthey

Labels
Abnahme ok Passt für die Milestoneversion bug Something isn't working Semantik Issues zu Semantik und strukturierte Daten
Projects
None yet
Milestone
V2.1 - Finetuning für Release
Development

No branches or pull requests
2 participants
@xwolfde @MManthey