scieloorg / scielo.org

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

prevenção de erro de comunicação com blog e twitter #91

Closed paratiuid closed 5 years ago

paratiuid commented 5 years ago

O que esse PR faz?

Foram adicionadas verificações de recebimento de xml, caso algum erro ocorra, uma mensagem amigável de erro deve ser exibida para os usuários no espaço do blog e twitter.

Onde a revisão poderia começar?

Pelos arquivos alterados que correspondem a exibição do blog na home do site.

Como este poderia ser testado manualmente?

Fiz um teste no meu servidor local alterando as keys de exibição de blog e twitter no arquivo config/constants.php

Algum cenário de contexto que queira dar?

Tivemos alguns erros de comunicação com o blog recentemente. Seja por firewall ou atualização do blog. Este ajuste visa evitar o aparecimento de erros php na tela. As mensagens devem aparecer de forma amigável.

Screenshots

Caso ocorra erros de comunicação com o blog, e twitter, devem aparecer mensagens amigáveis conforme print abaixo: Screen Shot 2019-04-12 at 2 58 52 PM

Screen Shot 2019-04-12 at 2 57 31 PM

Quais são tickets relevantes?

90

Referências

Prevenção de erros. Caso o xml seja inválido, retorne uma mensagem amigável de erro.

paratiuid commented 5 years ago

@joffilyfe acabei criando outro pull request na hora de enviar o ajuste.

https://github.com/scieloorg/scielo.org/pull/99

Incluí as constantes que estavam faltando. Acredito que agora o sistema deva subir corretamente.

joffilyfe commented 5 years ago

@paratiuid então este PR deve ser fechado?

paratiuid commented 5 years ago

@joffilyfe acredito que a solução para o problema de não conseguir subir o sistema esteja no arquivo que mandei no outro pr. Os dois estão relacionados. Tratam do mesmo problema. Apenas não consegui dar o push na mesma branch.

joffilyfe commented 5 years ago

@joffilyfe acredito que a solução para o problema de não conseguir subir o sistema esteja no arquivo que mandei no outro pr. Os dois estão relacionados. Tratam do mesmo problema. Apenas não consegui dar o push na mesma branch.

@paratiuid perguntei isso porque o outro PR contém todas as modificações deste, veja:

Screen Shot 2019-04-25 at 11 03 00
paratiuid commented 5 years ago

@joffilyfe exato! tratam do mesmo problema. Se for o mais correto a se fazer, pode desprezar este e foque na aprovação do outro onde enviei o arquivo contants.php corrigido.

joffilyfe commented 5 years ago

Substituído por #99