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

Adiciona mapa com as localizações #382

Closed
wants to merge 15 commits into from
Closed

Conversation

giovana-morais
Copy link
Contributor

@giovana-morais giovana-morais commented Sep 16, 2020

Descrição do PR
Inserção de um mapa interativo com a localização dos capítulos do PyLadies por todo o país.

Descrição das mudanças

  • adição de um script para gerar o mapa
  • adição de um script para buscar as informações dos capítulos pelo geopy
    • @biancarosa eu achei a api pra conseguir os dados de latitude e longitude. talvez dê pra gente fazer algo via bash ou sei lá e colocar no build do projeto.
  • adição de um popup em cada ponto do mapa com as informações do capítulo
  • remoção da lista de capítulos da página, deixando somente o mapa (<- vamos discutir esse ponto, pfvr)

Screenshot
Captura de tela de 2020-09-16 19-29-41

Captura de tela de 2020-09-18 18-44-01

Confirmações
Antes de enviar o Pull Request, faça as seguintes confirmações

  • O PR foi testado localmente
  • (se aplicável) Nenhum link está quebrado
  • (se aplicável) Nenhuma imagem está quebrada

Contexto adicional

  • Fiz o teste usando as logos dos capítulos como marcadores e ficou muito feio hahaha vamos ter que padronizar as imagens antes de conseguir usar isso de fato no mapa
  • Como eu re-escrevo o arquivo de locations.yml pela lib do python, ele fica meio esquisitinho (tem o lance das acentuações e tal). Ainda não testei se temos alguma opção de pretty print ou qualquer coisa assim pra ficar um pouco menos assustador pra quem vai adicionar um novo capítulo ao documento.
  • É possível que haja ambiguidade no nome das cidades. Talvez o capítulo de Sul de Minas se encaixe nisso. Nesses casos eu ainda não sei o que fazer, mas talvez revisar na mão.
  • Tentei manter o código todo em português. Por favor, chequem se ficou bom, é um primeiro teste também hahaha

Enfim, vamos conversar sobre isso. Olhem a preview. 👀

@giovana-morais giovana-morais changed the title [WIP] Adiciona mapa com as localizações Adiciona mapa com as localizações Sep 18, 2020
@giovana-morais giovana-morais requested a review from a team October 12, 2020 23:34
@giovana-morais
Copy link
Contributor Author

preciso de uma força com o css pra ficar bom de mergear isso aqui ):

@stale
Copy link

stale bot commented Dec 24, 2020

Essa issue foi marcada automaticamente como obsoleta porque não teve atividade recente. Ela será fechada se nenhuma atividade ocorrer nos próximos dias. Obrigada por suas contribuições.

@stale stale bot added the obsoleto label Dec 24, 2020
@stale
Copy link

stale bot commented Dec 31, 2020

Encerrando!

@stale stale bot closed this Dec 31, 2020
@jtemporal jtemporal deleted the feat/mapa_locais branch October 29, 2022 16:11
@jtemporal jtemporal restored the feat/mapa_locais branch October 29, 2022 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Discussão] Mapa na página de locais
2 participants