Skip to content

Conversation

penguinuux
Copy link
Contributor

@penguinuux penguinuux commented May 11, 2024

Este PR adiciona o mapa com as posições geográficas dos abrigos e localização do usuário.

Este PR trás as seguintes melhorias:

  • Exibe o mapa no topo da página (Leaflet + OpenStreetMap)
  • Á partir das latitudes e longitudes cadastradas no abrigo, seta pins com estas posições
  • Desde que o usuário autorize, faz o rastreamento em tempo real de sua posição e a exibe no mapa, junto a um círculo indicando a precisão calculada em metros (hook específico)
  • Renderiza novos pins de acordo com os filtros aplicados ou navegação efetuada
  • Exibe um botão que permite retornar para a posição atual do usuário (Pan)

Dependências adicionadas

  • leaflet
  • react-leaflet

Registro

image

@brds15
Copy link

brds15 commented May 11, 2024

ideia muito boa

@antoniolucasnobar
Copy link

LGTM, exceto pelos comentarios no componente Home.tsx, que, aparentemente, podem ser removidos.

@penguinuux
Copy link
Contributor Author

Comentários removidos.

@penguinuux penguinuux requested a review from sombraSoft May 11, 2024 17:12
Copy link
Contributor

@kelvinsb kelvinsb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, ficou bom!!

Talvez seja interessante no futuro adicionar mais informações no card do marcador(como título, endereço e status(lotado ou não)) e adicionar na página de detalhes do abrigo.

Copy link

@andersoncscz andersoncscz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top man 👏

@lucianomlima
Copy link
Member

@penguinuux pode fazer um rebase e resolver os conflitos?

@penguinuux
Copy link
Contributor Author

@penguinuux pode fazer um rebase e resolver os conflitos?

Positivo. Em andamento.

Co-authored-by: Luciano Lima <[email protected]>
@penguinuux
Copy link
Contributor Author

Resolve #172 se não totalmente, em sua grande maioria.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants