scieloorg / scielo.org

Website institucional da Rede SciELO
MIT License
1 stars 7 forks source link

Prevenção de erro de backend - mensagens amigáveis #144

Closed paratiuid closed 4 years ago

paratiuid commented 4 years ago

O que esse PR faz?

Verifica o status code do retorno do json. Caso seja diferente de 200, exibe uma mensagem de erro amigável Caso não seja possível reproduzir o erro, podemos forçar o status 500 removendo os comentários da linha 1264 no arquivo controllers/home.php.

Com isso, a mensagem de erro deve aparecer na tela.

Onde a revisão poderia começar?

Siga os passos descritos acima.

Como este poderia ser testado manualmente?

Siga os passos já citados.

Algum cenário de contexto que queira dar?

Consegui testar apenas forçando o status_code 500 via php, com a função http_response_code. Acredito que seja importante testarmos na homolog.scielo.org. Geralmente no fim de tarde consigo reproduzir o erro.

Screenshots

Screen Shot 2020-01-16 at 1 58 44 PM

Quais são tickets relevantes?

139

Referências

https://www.php.net/manual/pt_BR/function.http-response-code.php

paratiuid commented 4 years ago

@paratiuid

Por favor, alterar as mensagens de erro para:

EN: "An error occurred while searching. Please try again later." PT: "Ocorreu um erro durante a busca. Por favor tente novamente mais tarde." ES: "Ocurrió un error al buscar. Por favor intenta nuevamente más tarde."

Feito!

paratiuid commented 4 years ago

@joffilyfe

acabo de enviar o ajuste.