search

pokemongovet / pokemongovet.github.io

Projeto para incentivar adoção de animais pessoas que amam pokemons virtuais
https://pokemongovet.github.io/
Other
102 stars 37 forks source link

Página 404 e favicon. #63

Closed guilmour closed 7 years ago

fititnt commented 7 years ago

@evansantos @blackmoral @rcgivisiez dois de vocês podem testar aqui isso pra ver se podemos aceitar o PR?

rcgivisiez commented 7 years ago

Oi pessoal. @guilmour Testei o commit e há um pequeno problema com o redirecionamento. É preciso redirecionar para a index, e no momento ele redireciona para a raiz do servidor, o que é bem perigoso. Ficamos no aguardo da correção, obrigada! :)

guilmour commented 7 years ago

Oi, @rcgivisiez. Obrigado pela resposta. Muito legal o projeto e poder ajudá-lo. Então, tentei imitar o que ocorre aqui no gh-pages, duprando com um servidor simples http já na pasta do projeto; por isso penso ser o ideal. Se a raiz do site não for a próprio diretório do pokemongo.vet em um outro servidor, será preciso fazer alterações no .htaccess como o do github, que puxa o arquivo "404.html" sempre que há um 404. Por isso penso que não podemos redirecionar direto para "index.html", pois pode acontecer de o usuário chegar na página de 404 por meio de um link http://pokemongo.vet.br/textoqualquer/texto, por exemplo, e ser redirecionado para http://pokemongo.vet.br/textoqualquer/index.html. :disappointed: Se você tiver python aí, dá pra depurar fácil, direto no diretório com:

cd pokemongo.vet.br

wget raw.githubusercontent.com/guilmour/carlana/master/carlana.py 

python carlana.py

e acessar: http://localhost:8787/404.html

Por isso, nesse caso do gh-pages ou de o projeto estar na raiz do domínio, acho essa solução do PR apropriada. O que acham? :+1:

hsouza90 commented 7 years ago

Olá @guilmour, verifiquei que a alteração do favicon está apenas na index, não nas outras páginas, precisaria colocar em todas.

Realizei o teste (gostei desse carlana.py), não vi problema em redirecionar para a raiz (index também não fará diferença), realmente só funciona com host (local ou online), mas está ok.

Conforme verificado em: [1] https://help.github.com/articles/creating-a-custom-404-page-for-your-github-pages-site/ [2] http://mycyberuniverse.com/developing/custom-404-page-for-website-hosted-on-github.html

Comitando a página já estará com tudo funcionando.

hsouza90 commented 7 years ago

Bom, por mim fechou, @fititnt e @rcgivisiez, e vocês?

fititnt commented 7 years ago

@blackmoral @guilmour eu to meio enrolado com o https://github.com/fititnt/carta-aberta-premio-nacional-acessibilidade-na-web, mas vou revisar isso nesse sabado. Se eu não fizer, podem me marcar que o github vai apitar

fititnt commented 7 years ago

@guilmour Muito obrigado pelo seu PR! Já ta no ramo principal. Te explicar uma unica alteração que fiz, até porque isso não é algo intuitivo

https://github.com/pokemongovet/pokemongo.vet.br/commit/ca77926d9a14c1ce660a712e62b76d31a97a4a4d#diff-aa8eb2fe477b2d36ecc0f14d6422513fR38

Repare na diferença entre "/" e "./". O ponto significa relativo à pasta atual, enquanto a barra significa relativa a raiz do site. Isto é uma forma de resolver caminhos de arquivos sem precisar, por exemplo, usar uma linguagem backend que fica gerando as URLs