justarrived / just-match-frontend

[ABANDONED] Frontend for JustMatch API
https://antrop.invisionapp.com/share/FA62UU66R
MIT License
2 stars 9 forks source link

Zip code error on map #627

Closed FivelFred closed 8 years ago

FivelFred commented 8 years ago

On production, as a user who is not logged in the map puts the job in Demark. Turns out it sets the pin on a street called Sweden. Needs to be fixed ASAP.

post zip code error ends up in denmark

FivelFred commented 8 years ago

Not sure if this is back-end or front-end issue?

ti-sedp commented 8 years ago

I plotted makers from zip-latitude and zip-longitude field got it from api

2559-06-20_23h21_25

buren commented 8 years ago

Hm.. I need to investigate this a bit further. I tried one of the jobs in production and it displayed the job in Denmark, but when checking the coordinates I got some place outside of Kiruna (Sweden) ❓

I'm not doing anything fancy, I simply take the address fields and send them to Google and ask for the latitude, longitude coordinates.. I saw that I'm using a float type in the database to store these coordinates, so perhaps there are some rounding issues that causes this. I'll look into it ASAP.

buren commented 8 years ago

I've referenced an issue ☝️ that will fix this in the API, no need for any frontend changes.. Closing this in favour of https://github.com/justarrived/just_match_api/issues/450