SOS-RS
feat: buscar coordenadas do abrigo
🔨 Tenho uma nova PR para vocês revisarem 🤔 O que foi feito? Incluído integração com API do Google Maps para buscar coordenadas dos abrigos automaticamente quando eles são criados ou atualizados por admins. Por que? Acho interessante podermos pensar na possibilidade de mostrar a distância do usuário até o abrigo em uma futura funcionalidade no frontend. Talvez buscar os abrigos mais próximos dele. Buscar automaticamente as coordenadas através do Google Maps facilita o desenvolvimento dessa possível funcionalidade.
Importante:
-
Caso a API não esteja funcionando, ela não quebra as funcionalidades de Inclusão e atualização, apenas mantem os valores de longitude e latitude como null. Pelo que percebi esses valores não estão sendo informados na criação dos abrigos.
-
Sobre a utilização da API, ficaria feliz em contribuir com a disponibilização da API Key, se for possível e caso acatem a ideia.
📗 Checklist do desenvolvedor Foi testado localmente? Sim. Foi adicionado documentação necessária (swagger, testes e etc)? - Não
👀 Checklist do revisor Revisor 1️⃣ Você entendeu o propósito desse PR? Você entendeu o fluxo de negócio? Você entendeu o que e como foi desenvolvido tecnicamente a solução? Você analisou se os testes estão cobrindo a maioria dos casos?
@wOL-Lucas ótima idéia, deixei um comentário...
Tenho mais 2 pontos, os testes unitários e padrão de nomenclaturas (CamelCase vs PascalCase)) mas devido a urgência do propósito da aplicação, não sei como estamos lidando aqui (vou deixar essa dúvida no Discord).
@wOL-Lucas mais uma dúvida... ainda não consegui subir o projeto então não sei como está hoje, mas e caso a latitude e longitude já tenham sido preenchidos? Não vale verificar se eles estão nulos antes de seguir no fluxo de busca automática?
@thaua, pelo que verifiquei hoje em prod mesmo essas informações não estão sendo preenchidas, mas obviamente é uma boa ideia manter o input original. Não me atentei nessa questão.
Vou aproveitar as outras alterações que você propôs e ajustar.