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

Standort - Datenprüfung und Distanzberechnung #12

Open
vMarkus opened this issue Apr 14, 2020 · 4 comments
Open

Standort - Datenprüfung und Distanzberechnung #12

vMarkus opened this issue Apr 14, 2020 · 4 comments
Assignees

Comments

@vMarkus
Copy link

vMarkus commented Apr 14, 2020

Transportiert aus Frontend

Idee

via Google Maps realisieren enstelle des aktuell angebundenen Service?

@marcoebbinghaus
Copy link
Contributor

Ich sehe keinen Vorteil darin, jetzt von LocationIQ auf GoogleMaps zu wechseln. Was sind die Argumente für einen Wechsel? Die falsche Distanz in remedyMatch/rm-frontend#20 ist definitiv nicht auf die Datenqualität von LocationIQ zurückzuführen, sondern einfach ein Bug irgendwo bei uns im Code (vermutlich falsche Geokoordinaten im Standort hinterlegt, weil die Adresse unvollständig eingegeben wurde). Falls das der Grund für den vorgeschlagenen Wechsel ist, ist also kein Wechsel nötig.

@vMarkus
Copy link
Author

vMarkus commented Apr 14, 2020

Das Argument ist, dass wir den Umkreis der Spenden/Bedarfe begrenzen (50km?) - das kann einige Matches verhindern.

Workarounds

  1. Umkreistoleranz erhöhen
  2. Adressqualität erhöhen (z. B. RegEx/Java für PLZ, Ort, Straße/Hausnummer - soweit sinnvoll für Erstausfüllung der Profile und Standorte. Evtl. Wiederverwendbarkeit aus dem Registrierungsformular, falls denn umgesetzt)

LocationIQ Umstieg auf Google Maps kann durchaus erst später erfolgen, bin beim Locationservice leidenschaftslos

@marcoebbinghaus
Copy link
Contributor

Bezug nehmend auf remedyMatch/rm-frontend#20 : Kann mir jemand sagen wie das reproduzierbar ist? Wo gebe ich welchen Standort ein um auf die genannte fehlerhafte Distanz (40 statt 15-20km) zu kommen?

@vMarkus
Copy link
Author

vMarkus commented Apr 16, 2020

info@remed... user - Eintrag 63683 Ortenberg über "zusätzlicher Standort"
sowie
markus.hackathon@gm... Eintrag 63654 Büdingen als primare Adresse

Aktuell sind es 30 km, an sich aber deutlich weniger

Screenshot anbei

Screenshot 2020-04-16 at 18 34 54
Screenshot 2020-04-16 at 18 36 15

@marcoebbinghaus marcoebbinghaus self-assigned this Apr 17, 2020
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