ThundeRatz / ThunderSite

🌐 Website with django
https://thunderatz.org
2 stars 1 forks source link

Fix D'Arc project image #12

Closed FelipeGdM closed 4 years ago

FelipeGdM commented 5 years ago

Fala pessoal! Tudo certo?

Entonces, já faz um tempo que a nossa queria D'Arc está sem foto no site.

Eis que, após algumas horas debugando o site e o banco de dados com o @Drebt, pudemos descobrir que o banco de dados está correto (a imagem existe e é acessível) e que o problema estava no código django python.

Alterei a forma como a imagem é puxada pelo python, de forma a deixar a cargo do css carregar a foto, do mesmo modo que é feito com a imagem de background dos cards dos robôs

Com essa mudança, é possível consertar a foto de perfil da D'Arc \o/

No mais, fiquem bem e mantenham-se hidratados (especialmente nesse tempo seco >.< )

FelipeGdM commented 4 years ago

Fala pessoal!

Cá estamos novamente para solucionar o problema da foto da nossa querida D'Arc. Rodando o site localmente, é possível isolar exatamente o problema: a biblioteca lazy loading do javascript não consegue carregar a foto da D'Arc por causa do apóstrofe. A imagem da D'Arc é salva na pasta /media/projects/D'Arc/card_Darc2.jpg e a biblioteca não consegue carregar imagens em endereços que contém caracteres especiais, como o '

Discuti ideias com o @Drebt a respeito de como poderíamos consertar isso. Fuçar a biblioteca javascript mencionada pelo @d-nery era uma opção, mas mó rolê e javascript, né.

A nossa ideia é mudar o caminho no qual são salvas as imagens do projeto. Ao invés de usar o nome do projeto no endereçamento, utilizar o slug associado. O slug identifica o projeto da msm forma que o nome, só que sem caracteres especiais. É o jeito mais "safe" de indentificar as coisas.

No mais, é isso. Funcionou qdo eu rodei localmente, agora é só produzir em massa

d-nery commented 4 years ago

Ou, algo aconteceu aqui, esse monte de commit de merge não devia tá aí A base branch devia ser a develop dele, não a master (e a develop dele devia estar em dia com a develop nossa)