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

WIP Migrar vcge 1 vcge 2 #29

Closed
wants to merge 7 commits into from
Closed

WIP Migrar vcge 1 vcge 2 #29

wants to merge 7 commits into from

Conversation

caduvieira
Copy link
Contributor

O arquivo do VCGE 2.0.3 referencia itens do VCGE 1 que não existem.

ericof and others added 2 commits September 20, 2016 14:50
Atualiza changelog

Adiciona aviso sobre a nova versão do VCGE

Altera o teste para utilizar assertIn ao inves de assertTrue

Atualiza versao do vcge.n3

Modificados arquivos de testes que continham links para endereços da antiga versão do VCGE. Basicamente substituido #achados-perdidos para #cultura

Correção de uma id no arquivo n3. Havia acentuação.

Correção do número de termos atuais do VCGE nos arquivos de teste.

Atualiza com arquivo do dia 21 de maio de 2014

Remove acento

Modificado URI base do arquivo n3.

Atualização dos arquivos CONTRIBUTORS.rst, CHANGES.rst e setup.py, com informações pertinentes a esta nova versão do plugin.

Follow ZCML style guide

Register package

This is to avoid an issue with package not installing when profiles are hidden.

See: plone/plone.app.event#159

Substituição do arquivo vcge.n3 de versão 2.0.2 para 2.0.3

Preparing release 2.0.0

Back to development: c

Fix version

Fix changelog

Add Pillow as a testing dependency on CI

Instala egg pexpect explicitamente

Quando o ipython é instalado com buildout, o pexpect não é instalado
como dependência. Ele deve ser baixado explicitamente, até que a issue:
ipython/ipython#8806
seja resolvida.

(closes) #30

Remove iw.debug

Evita erro:

ImportError: No module named pexpect

Ver:
ipython/ipython#8806

close #31
@caduvieira
Copy link
Contributor Author

caduvieira commented Sep 20, 2016

Estou em dúvida como prosseguir. Após o upgradeStep seria a instalação de uma nova versão (2.0.1) do pacote? Fiz um squash com os commits do master (versão 2.X.X) mas agora fiquei na dúvida

Estava esperando que a versão mais nova do VCGE (2.1.0) saísse com o dcterms:replaces mas não saiu até agora...

@idgserpro
Copy link
Member

@caduvieira você tem que fazer rebase com o master. Mas essa branch está muito diferente do master. Vai ser dificil fazer rebase. Talvés o melhor seja excluir essa branch o começar de novo.

@coveralls
Copy link

Coverage Status

Coverage decreased (-48.5%) to 43.243% when pulling 5c66e93 on migrar_vcge_1_vcge_2 into 0b38068 on 1.0.x.

@augusto-herrmann
Copy link

Não existe VCGE 1. Existe VCGE 2011 e existe VCGE 2. Os commits mencionam um "upgrade para versão 2010". Não entendi.

Além disso, o upgrade será opcional para o administrador do site? Pergunto pois o VCGE 2, por ser muito reduzido, particiona muito menos os itens em assuntos, dificultando a recuperação da informação em sites com muito conteúdo.

@idgserpro
Copy link
Member

idgserpro commented Sep 21, 2016

@augusto-herrmann provavelmente o VCGE 1 aqui do produto é o VCGE 2011 que você falou. Esse aqui.

2010 é a versão do profile do produto e não do VCGE.

A branch master é o VCGE 2 e a branch 1.0.x é o VCGE 1. Então se o usuário está utilizando uma versão 2.x do brasil.gov.vcge ele está utilizando o VCGE 2 e se ele está utilizando a versão 1.x do brasil.gov.vcge ele está utilizando o VCGE 1.

Se ele utilizava a versão 1.x do brasil.gov.vcge e atualizou para versão 2.x do brasil.gov.vcge ele passou a utilizar o VCGE 2 e deveria ter um upgrade para atualizar o conteúdo.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 91.634% when pulling fc46415 on migrar_vcge_1_vcge_2 into 0b38068 on 1.0.x.

@idgserpro
Copy link
Member

idgserpro commented Sep 21, 2016

@caduvieira você tem certeza que esse upgrade deveria ser feito na branch 1.0.x? A branch 1.0.x utiliza VCGE 1. Não deveria ter um upgrade nela para atualizar para VCGE 2. Esse upgrade deveria ser feito no master.

@caduvieira
Copy link
Contributor Author

caduvieira commented Sep 21, 2016

@augusto-herrmann o quer eu quero referenciar é a versão do egg do brasil.gov.vcge 1.0.x para brasil.gov.vcge 2.0.x e não a versão do VCGE em si. E seria opcional sim.

@idgserpro pensando aqui eu acho que deveria ser na branch 1.0.x mesmo. Seria um upgradeStep que faria três coisas:

  • Atualizar os valores dos contents que possuam o VCGE
  • Instalar o egg vcge 2.0.x
  • Desinstalar o egg vcge 1.0.x

O que você acha?

O que não estou entendendo direito é por que seria na master se a master já está na versão 2.0.x.

@idgserpro
Copy link
Member

@caduvieira não é possivel "instalar" e "desinstalar" eggs com upgrade step. Um usuário que está utilizando a versão 1.0.x do brasil.gov.vcge está utilizando VCGE 1. Se ele deseja utilizar o VCGE 2 nesse site, ele precisa ir no buildout e pinar a versão 2.0.x do brasil.gov.vcge. Ai precisaria ter um upgrade step que:

  • Verifique que ele estava utilizando o VCGE 1
  • Atualiza os conteúdos para VCGE 2

Esse upgrade deve ter o source="*" porque a qualquer momento o usuário pode querer migrar do VCGE 1 para o VCGE 2.

Por isso eu acho que esse upgrade deveria ser na branch master.

@caduvieira
Copy link
Contributor Author

Entendi! Não sabia dessa limitação. Vou refazer esse branch partindo do master então. Obrigado

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.

6 participants