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

Adiciona atualizações mysql das versões 4.7.1 e 4.7.2 #252

Merged
merged 5 commits into from
May 16, 2017

Conversation

rafaelpatro
Copy link
Contributor

@rafaelpatro rafaelpatro commented May 12, 2017

Incluídos os arquivos de atualização de versão para os releases 4.7.1 e 4.7.2
Removidos os códigos de serviços inválidos desde 05/05/2017
Adicionado novo PAC GF (04693) e respectiva validação
Adicionada Carta aos serviços pré-instalados

Closes #251

Copy link
Owner

@pedro-teixeira pedro-teixeira left a comment

Choose a reason for hiding this comment

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

Não existe 4.7.3, só 1 e 2.

Aliás, deveria ser:

  • upgrade-4.7.0-4.7.1.php: só uma migração vazia
  • upgrade-4.7.1-4.7.2.php: outra migração vazia
  • Renomear upgrade-4.7.0-4.8.0.php para upgrade-4.7.2-4.8.0.php
  • Manter upgrade-4.8.0-4.8.1.php vazia

Não?

@@ -28,24 +28,6 @@
$installer->getConnection()->insertForce(
$this->getTable('pedroteixeira_correios/postmethod'),
array(
'method_id' => '1',
Copy link
Owner

Choose a reason for hiding this comment

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

Não é mais usado?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sobre os códigos sem contrato...

  • Os códigos (41106 e 40010) foram substituídos pelos novos (04510 e 04014).
  • Na PR anterior, mantive os antigos e novos, pois ainda funcionavam.
  • Agora os antigos deixaram de funcionar.
  • Sugiro retirar pois já apareceram reclamações de mal funcionamento Versão 4.8.0 simplesmente não funciona #251, e aparentemente o problema foi que o usuário escolheu os códigos antigos.

Copy link
Owner

Choose a reason for hiding this comment

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

Ok

@rafaelpatro
Copy link
Contributor Author

Realmente os arquivos de versões ficaram errados. Foi a pressa :/
Vou corrigir assim q possível.

@rafaelpatro rafaelpatro changed the title Adiciona atualizações mysql das versões 4.7.2 e 4.7.3 Adiciona atualizações mysql das versões 4.7.1 e 4.7.2 May 14, 2017
oliva1337 added a commit to oliva1337/correios that referenced this pull request May 15, 2017
@pedro-teixeira pedro-teixeira self-assigned this May 15, 2017
@pedro-teixeira pedro-teixeira added this to the v4.8.1 milestone May 15, 2017
@pedro-teixeira pedro-teixeira merged commit e748726 into pedro-teixeira:master May 16, 2017
pedro-teixeira added a commit that referenced this pull request May 16, 2017
pedro-teixeira added a commit that referenced this pull request May 16, 2017
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.

None yet

3 participants