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

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

@pedro-teixeira pedro-teixeira May 13, 2017

Não é mais usado?

Copy link
Contributor Author

@rafaelpatro rafaelpatro May 14, 2017

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 #251, e aparentemente o problema foi que o usuário escolheu os códigos antigos.

Copy link
Owner

@pedro-teixeira pedro-teixeira May 14, 2017

Ok

@rafaelpatro
Copy link
Contributor Author

@rafaelpatro rafaelpatro commented May 14, 2017

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 issue 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
1 check passed
pedro-teixeira added a commit that referenced this issue May 16, 2017
pedro-teixeira added a commit that referenced this issue 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
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants