search

plonegovbr / brasil.gov.agenda

Pacote de Agenda para Membros do Governo Brasileiro
https://plone.org.br/gov/
3 stars 2 forks source link

Upgrade steps pendentes #108

Closed hvelarde closed 6 years ago

hvelarde commented 6 years ago
idgserpro commented 6 years ago

Essa issue é mais pra controle? Você fará a adição de upgradeStep no mesmo PR da remoção em https://github.com/plonegovbr/brasil.gov.agenda/pull/107 ou outro PR?

hvelarde commented 6 years ago

em outro; estamos com deadlines e algumas coias não estão bem definidas ainda.

idgserpro commented 6 years ago

Esse upgradeStep tem de ser bem estudado, pois collective.portlet.calendar adiciona uma browserlayer, provavelmente será necessário algo semelhante a https://github.com/plonegovbr/brasil.gov.portal/blob/152183c01d679ac5bc4b4bd64f3c5156aa3ad6e8/src/brasil/gov/portal/bbb.py.

hvelarde commented 6 years ago

o browser layer é removido na desinstalação do pacote: https://github.com/collective/collective.portlet.calendar/blob/1.0b3/collective/portlet/calendar/tests/test_setup.py#L49-L52

idgserpro commented 6 years ago

Sim, mas você não chamou a desinstalação do pacote. O ideal às vezes teria sido uma versão intermediária na 2.x que desinstala o pacote e numa mais nova remove do buildout como dependência. Como o PR já foi mesclado, talvez o a ser feito é uma última versão então do 1.5.x ter um upgradeStep que chama o profile de uninstall. Aí, quando instalar a 2.x, onde já não tem referência pro egg, não vai dar erro.

hvelarde commented 6 years ago

não, a desinstalação de pacotes tem que ser manual; não sabemos se os portlets estão sendo usados em outra coisa além da agendas.

acho que está claro na seção de atualização; o problema é que as pessoas não leem ;)

hvelarde commented 6 years ago

não vamos remover nenhum portlet via upgrade step; o pacote foi reintroduzido como dependência para corrigir um problema com o browser layer (#137).

por outro lado, foi adicionado um script que mostra os portlets de um portal; isso pode servir para remover eles antes ou depois da migração:

https://github.com/plonegovbr/portal.buildout/commit/12f074109c5d918c53d0689c787514cd4feeb5b6

idgserpro commented 6 years ago

Com relação a https://github.com/plonegovbr/portal.buildout/commit/12f074109c5d918c53d0689c787514cd4feeb5b6, uma melhoria seria ter uma browser view pro próprio usuário final acessar e poder corrigir administrativamente essas associações.

hvelarde commented 6 years ago

@idgserpro o collective.portlet.calendar nem está instalado; essa é a parte estranha.