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

Limitar buscas fora de SP, Capital #7

Open
1 of 3 tasks
bplmp opened this issue Apr 9, 2018 · 5 comments
Open
1 of 3 tasks

Limitar buscas fora de SP, Capital #7

bplmp opened this issue Apr 9, 2018 · 5 comments
Assignees
Labels
enhancement New feature or request prioridade-média

Comments

@bplmp
Copy link
Contributor

bplmp commented Apr 9, 2018

  • Mostrar mensagem de erro para buscas de endereços fora de SP, Capital
  • Limitar buscas pela API do Google
  • Validar o input do usuário por bounding box da cidade de SP, após retorno do endereço georeferenciado
@bplmp bplmp added the enhancement New feature or request label Apr 9, 2018
@bplmp bplmp self-assigned this Apr 9, 2018
@bplmp
Copy link
Contributor Author

bplmp commented Apr 10, 2018

874f269 Limita a API do Google Places Autocomplete para Brasil, mas ainda não limita só para cidade de são paulo.

Acompanhar essa issue para ver quando será suportado limitar por cidade: hibiken/react-places-autocomplete#143

@victorsenam
Copy link
Contributor

Parece que a API não suporta mais o strictBounds. Essa aqui parece ser a versão atualizada da API, a versão com o strict bounds é essa, que tem um aviso de deprecação no topo da página.
Além, disso, me parece que o location e o radius são ignorados já que o bounds está definido.

@bplmp
Copy link
Contributor Author

bplmp commented Sep 19, 2018

@victorsenam acho que não tem o que fazer então. Vamos só remover o location e o radius pra limpar o componente então?

@victorsenam
Copy link
Contributor

Se a gente decidir que isso é realmente bem importante, a gente pode tentar costruir uma restrição estrita em cima do retorno da API. Mas isso deve dar algum trabalho. Topo limpar o componente.

@bplmp
Copy link
Contributor Author

bplmp commented Sep 25, 2018

Acho que não precisamos fazer essa restrição em cima da API. Afinal a gente já pega os erros no front end. O único ganho seria nem mandar o request pra API se está fora dos bounds, as não vejo muita vantagem por enquanto. Vamos limpar o componente.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request prioridade-média
Projects
None yet
Development

No branches or pull requests

3 participants