Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 479 imagem destaque nao renderiza #481

Merged
merged 4 commits into from
Jun 8, 2018

Conversation

idgserpro
Copy link
Member

Boas práticas para o PR: Checklist

Foi aberta uma issue relativa a esse PR;

  • Sim
  • Não
  • Não se aplica

Foram adicionados testes unitários;

  • Sim
  • Não
  • Não se aplica

Foram adicionados testes robots;

  • Sim
  • Não
  • Não se aplica

Foram adicionados upgradeSteps;

  • Sim
  • Não
  • Não se aplica

Foi adicionada a modificação no CHANGES.rst do pacote, sempre no topo do arquivo (ou seja, na primeira linha do release ainda não lançado), contendo o seu nome de usuário do github e a referência ao issue que você está tratando (não esqueça de colocar no fim do arquivo a url para a issue, use o padrão do próprio arquivo);

  • Sim
  • Não
  • Não se aplica

PR não contém assuntos diferentes. PR's devem ter um objetivo claro para facilitar o review. Ex: não junte, num mesmo PR, alterações de code-analysis e de implementação. Faça um PR de code-analysis e depois o da implementação em si;

  • Sim
  • Não
  • Não se aplica

Evite muitos commits pequenos de um mesmo assunto no PR, sempre que possível efetue rebase;

  • Sim
  • Não
  • Não se aplica

idgserpro added 3 commits June 1, 2018 15:49
Imagem de destaque opcional acima da barra de destaques volta a ser
renderizada. Esse recurso está disponível como mencionado em

http://www.secom.gov.br/orientacoes-gerais/comunicacao-digital/gestao-manual-portal-modelo-governo-federal-dez2014.pdf#page=226

Resolve o relato #479
Testa além dos links de destaque a possibilidade de adicionar um tile de
texto rico na mesma capa de destaques.
Por algum motivo, "Compose Cover" nesse contexto dá timeout:

https://travis-ci.org/plonegovbr/brasil.gov.portal/builds/387898576#L1976

Mudei a chamada para outra keyword pois em navegadores mais novos esse
erro com essa keyword não ocorre.
@idgserpro idgserpro requested a review from hvelarde June 7, 2018 17:28
@idgserpro
Copy link
Member Author

Após a mescla desse PR, podemos seguir #478 para gerar os releases dos pacotes e um release 1.5.2 em portalpadrao.release.

Copy link
Member

@hvelarde hvelarde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Independentemente da visão da capa de "Destaques", os tiles ali
adicionados devem ser renderizados.
@idgserpro idgserpro force-pushed the issue-479-imagem-destaque-nao-renderiza-idgserpro branch from 086ff43 to 1e46983 Compare June 8, 2018 11:21
@idgserpro idgserpro merged commit 45c5f6f into 1.x Jun 8, 2018
@idgserpro idgserpro deleted the issue-479-imagem-destaque-nao-renderiza-idgserpro branch June 8, 2018 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants