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

Corrige ícones de tiles no overrides.zcml. #141

Merged
merged 2 commits into from
Apr 7, 2016
Merged

Corrige ícones de tiles no overrides.zcml. #141

merged 2 commits into from
Apr 7, 2016

Conversation

idgserpro
Copy link
Member

Corrige #140

@hvelarde Precisamos de uma nova release do collective.cover pra essa build passar. @rodfersou poderia dar uma olhada? O pull request está bem simples.

@rodfersou
Copy link
Member

@idgserpro vou esperar o release do collective.cover para aceitar o pull request.

@hvelarde
Copy link
Member

hvelarde commented Mar 1, 2016

@idgserpro posso ver esta tarde só.

@idgserpro
Copy link
Member Author

Ok! Se tiver uma previsão pro release do collective.cover, pode informar aqui mesmo. Com esse novo release já podemos fechar o plonegovbr/brasil.gov.portal#265

@hvelarde
Copy link
Member

hvelarde commented Mar 1, 2016

@idgserpro vou tentar fechar esta semana mesmo.

@rodfersou
Copy link
Member

@idgserpro poderia atualizar a versão do c.cover para a b1 neste pull request?

@idgserpro
Copy link
Member Author

@rodfersou você quer dizer com relação à pinagem em buildout.cfg onde está collective.cover = >= 1.0a13?

@rodfersou
Copy link
Member

@idgserpro sim, a versão nova não é 1.0a13, e sim 1.0b1

@idgserpro
Copy link
Member Author

@rodfersou foi preciso alterar um bocadinho de coisa pra rodar no 1.1b1, mas agora os testes estão passando.

# Após o commit 007adcde40d2031a15debd5122af8086a1db6fa6, necessito a
# partir dessa versão pois alterei o caminho dos recursos estáticos no
# overrides.zcml.
'collective.cover >= 1.0a13',
Copy link
Member

Choose a reason for hiding this comment

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

seria 1.0a12 (1.0a13 não existe)

@idgserpro
Copy link
Member Author

@rodfersou sugestões efetuadas.

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.

3 participants