feat: added hook to get current user geo position #134
Conversation
| import { GeoLocation } from './types'; | ||
| import { toast } from '@/components/ui/use-toast'; | ||
|
|
||
| const mockedUserLocation = { |
There was a problem hiding this comment.
Essa informação talvez seja só útil para fins de testes, se o usuário não aceitar compartilhar sua localização vai ficar definido esse harded-coded.
There was a problem hiding this comment.
imagino que mesmo se o usuário não aceite compartilhar a localização da para estipular sua localização aproximada utilizando seu ip(e avisar o usuario que isso esta sendo feito através das informações sobre os cookies🤔 )
There was a problem hiding this comment.
Isso, essa informação está ai pra fins de teste apenas, eu não estou no RS então fica dificil de validar.
There was a problem hiding this comment.
imagino que mesmo se o usuário não aceite compartilhar a localização da para estipular sua localização aproximada utilizando seu ip(e avisar o usuario que isso esta sendo feito através das informações sobre os cookies🤔 )
Dá sim, dependendo de aonde estão usando para subir o projeto dá pra pegar com um request no servidor. Só que acho que não seria de todo útil, Geocoding por IP é altamente impreciso, e acho que essa feature demanda mais refinamento. Acho que uma forma válida seria eventualmente permitir ao usuário de colocar o endereço em que se encontra, e rodar um Geocoding por trás deste endereço pra fazer a busca.
There was a problem hiding this comment.
O local padrão pode ser o estado do Rio Grande do Sul como um todo. Aqui tem alguma outra coisa que precisa de mudança? Por que para mim parece aceitável.
| import { GeoLocation } from './types'; | ||
| import { toast } from '@/components/ui/use-toast'; | ||
|
|
||
| const mockedUserLocation = { |
There was a problem hiding this comment.
Minha sugestão é fazer uso das ferramentas de desenvolvimento dos navegadores, eles já disponibilizam tooling pra simular localização (no Chrome por exemplo, basta abrir a dev tool, Ctrl + Shift + P e digitar "show sensors")
Primeiro passo para obter a lista de shelters por proximidade (ver issue #93). Esse hook facilita o acesso à API de Geolocalização do browser. É só usar o request quando o usuário clicar um botão (ou algum lifecycle) e ele já atualiza o estado interno com todas as validações, incluindo um toast caso dê erro.