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

Maybe custom component to map cities to Bundesländer? #35

Open
lislis opened this issue Jun 11, 2024 · 12 comments
Open

Maybe custom component to map cities to Bundesländer? #35

lislis opened this issue Jun 11, 2024 · 12 comments

Comments

@lislis
Copy link
Member

lislis commented Jun 11, 2024

Im Excel hat es jemensch geschafft eine Liste mit Städten zu ihren Bundesländern zu mappen. dh wenn mensch eine Stadt ausgewählt hat wurde in der Spalte Bundesland das dementsprechend ausgefüllt.

Wie schwer wäre das als Stapi plugin umzusetzen? Die Listen haben wir im Excel, es geht darum das als nices Interface umzubauen?

@andneuma
Copy link
Collaborator

andneuma commented Jun 18, 2024

@lislis Also du meinst, dass automatisch das Bundesland ausgefüllt wird auf Grundlage der Stadt? Instinktiv erster Gedanke wär im Backend Nominatim anzufunken, dort die Stadt reinzuwerfen und aus dem JSON blob den federal state rauszufischen. Das würde denke ich easy über Lifecycle hook funktionieren. Man kann das sicherlich auch hardcoden, spart Serveranfragen aber ist vlt etwas mehr arbeit #ecology

Die spannende Frage dann wäre noch ob quasi live die entsprechenden Bundesland Felder gefüllt werden sollen oder ob das via Backend dann onCreate / onUpdate reicht.

Edit: Grad ma gegooglt, die csv dafür gäbe es hier. Vlt kann man in Strapi zur Laufzeit ein Objekt damit füllen, auf das die entsprechenden React Components zugreifen können. Klingt nach ner spannenden Aufgabe für @kjoscha 😸

Edit2: Grad nochma drüber nachgedacht. Da Städtenamen ja nicht eineindeutig sind braucht es zusätzlich ja noch die PLZ. Ich frag mich ob sich der Aufwand für uns lohnt an der Stelle ein convenience feature zu bauen wenn mans doch auch in 1 Min via GMaps/OSM rausgefunden hat?

@lislis
Copy link
Member Author

lislis commented Jun 19, 2024

ich mein, wir können auch die Felder (Bundesland, Land, etc) aus der UI entfernen und sagen, alles andere wird nach dem speichern automatisch gefüllt, und uns das live update der Felder sparen? Ob der lookup dann über die CSV oder Nominatim läuft ist mir eher egal

@andneuma
Copy link
Collaborator

@lislis Also so, dass Leute dann nur Stadt und PLZ und vlt noch Straße etc angeben?

@lislis
Copy link
Member Author

lislis commented Jun 20, 2024

genau!

@andneuma
Copy link
Collaborator

ok, kann ich machen

@andneuma
Copy link
Collaborator

andneuma commented Jun 21, 2024

@lislis Ah nee, dis wird schwierig glaube ich weil PLZ nicht eingteragen wird womit wir dann kein eindeutiges identifier paar haben. gibt nur freitext. Meine Befürchtung wäre, dass zudem leute sich verschreiben

image

https://forum.strapi.io/t/lifecycle-and-components-inside-collections-type/14003/13

@lislis
Copy link
Member Author

lislis commented Jun 21, 2024

hmm, also wäre plz als Feld hinzuzufügen deiner Meinung nach nicht genug? Was wäre die alternative?

@andneuma
Copy link
Collaborator

achso, doch, missverständnis! also du meinst das plz feld würde noch dazu kommen? weil dann reichts, klaro! dh die information gibts eigentlich immer, nur das bundesland rauspuzzlen is so ein nerviger extra step? also sehe auf jeden fall, dass das sinn macht. kann das feld dann einfach mit dazu nehmen und via regex auf german postal code format einschränken

@andneuma
Copy link
Collaborator

andneuma commented Jun 27, 2024

ok, also das scheint irgendwie komplizierter als gedacht und mir schleierhaft wie man das gescheit in Strapi macht ehrlich gesagt :/ Das Problem liegt im Update von properties innerhalb von single / repeatable components. Vlt merken wir uns das einfach, v5 ist in der Betaphase und geht das Problem vielleicht an? Weird, ist für mein Empfinden voll basic eigentlich, dass genestete Objekte vor dem DB commit nochmal verändert werden können aber dafür steck ich zuwenig drin

@lislis
Copy link
Member Author

lislis commented Jun 27, 2024

okay! dann überleg ich mir mal wie ich erkläre warum das in der Excel einfach ging und in strapi nicht 😅 #computerthings

@andneuma
Copy link
Collaborator

andneuma commented Jun 27, 2024

viel erfolg 😅 aber vlt bevor du das machst: ich frag nochmal im strapi forum explizit nach, vlt kommt da ja noch was bei rum. solang können wir das issue ja offen lassen

Edit: Hab mal nachgehakt

@lislis
Copy link
Member Author

lislis commented Jul 9, 2024

Hier auch ein kleiner Pivot-Vorschlag: Anstatt des automatisierten mappings von Stadt auf Bundesland, zumindest die Stadt/Landkreis Liste aus dem Excel in ein Dropdown zu importieren, das als relation zu nutzen?
Damit wir auf jeden Fall erstmal vermeiden, dass Stadt/LAndkreis Freitextfeld ist und damit sehr fehleranfällig.
@andneuma können wir dein import-script vereinfacht dafür nutzen?

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

No branches or pull requests

2 participants