TINF-296 + TINF-272: Implementieren des Nominatim Requests + Encoding der URL

happy-hamster / backend

RESTful Spring Boot API

https://api.happyhamster.org

Apache License 2.0

2 stars 0 forks source link

Closed Mavv3006 closed 4 years ago

jstsddg commented 4 years ago

@Mavv3006 Bitte merge conflicts lösen!

Mavv3006 commented 4 years ago

TINF-272 können wir auch getrennt implementieren, aber url.replace() gehört da glaube ich nicht hin.

Musste ich machen, weil im Datenmodell der Query nicht mehr als blanker String, sondern als Set abgespeichert wird.

jstsddg commented 4 years ago

Du baust dir doch vorher schon deine URL mit , getrennt. Rein logisch solltest du dort ja annehmen können, dass deine URL korrekt gebaut ist.

Mavv3006 commented 4 years ago

Ich habe eine Idee wie ich es für diesen PR machen kann, um die Encoding dann in TINF-272 zu implementieren.

Mavv3006 commented 4 years ago

Erst wenn dieser PR gemerged ist, kann ich mir TINF-272 anfangen.

jstsddg commented 4 years ago

Aber so oder so ist die Zeile this.url = this.url.replace(" ", ","); unnötig, bzw. gibt Fehler potential

Mavv3006 commented 4 years ago

@MrKraboom Besser?

Mavv3006 commented 4 years ago

Ich mache die TINF-272 mit hier rein.

jstsddg commented 4 years ago

Ups, sorry :D