pyladies-brazil / br-pyladies-pelican

Site PyLadies Brasil usando Pelican
http://brasil.pyladies.com/
MIT License
69 stars 104 forks source link

fix load of yaml file #422

Closed gabicavalcante closed 3 years ago

gabicavalcante commented 4 years ago

Descrição do PR Esse PR vai corrigir a issue #414. Além da página de dojo, tbm tem a página de tutoriais que estava vazia e dando o mesmo erro. Corrigi tbm warning de leitura de arquivos yaml, a forma como estava sendo lida era com load, mas tá deprecated, e a melhor forma de fazer é com load_safe.

Descrição das mudanças A principal mudança é que agora tem uma verificação para checar se os arquivos de vídeos existem, e caso exista, vai usar somente os itens que tiverem um valor para o campo de url. Assim a gente garante que não vai ter um link not found. Além disso coloquei para que no template só mostre as seções com conteúdo. Então se não tiver links para dojos, nem aparece essa opção no template.

Screenshot Como ficou a tela de vídeos. pyladies-videos

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