meetnearme / api

1 stars 0 forks source link

Correctly handle missing `lat` / `lon` in home page fallback #112

Open brianfeister opened 1 month ago

brianfeister commented 1 month ago

The current default is (approximately) Denver or so, we either want to default to NYC or some primary location, OR render a different partial for "No location" that features a few different cities.

Defaulting to Denver here is definitely not the way, open to options about the right way

https://github.com/meetnearme/api/blob/d09bac1788759eb4298f439e0a277070762dff22/functions/gateway/handlers/page_handlers.go#L77-L82