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

Correção para Produtos Configuráveis #265

Merged
merged 5 commits into from May 28, 2017

Conversation

Projects
None yet
3 participants
@rafaelpatro
Contributor

rafaelpatro commented May 27, 2017

Atualmente ao trabalhar com Produtos Configuráveis, a extensão carrega os atributos incorretamente, compremetendo a cotação do frete.

Esta PR propõe modificar o comportamento do módulo em relação aos Produtos Configuráveis, carregando os atributos corretamente:

  • Permite trabalhar com Produtos Configuráveis, associando Produtos Simples de diferentes tamanhos e pesos.
  • Os atributos do módulo tornam-se aplicáveis apenas para Produto Simples e Produto Agrupado.

closes #244
closes #142
closes #111
closes #34

@pedro-teixeira pedro-teixeira self-assigned this May 27, 2017

@pedro-teixeira pedro-teixeira added this to the v4.9.0 milestone May 27, 2017

@@ -15,7 +15,7 @@
<config>
<modules>
<PedroTeixeira_Correios>
<version>4.8.1</version>
<version>4.8.2</version>

This comment has been minimized.

@pedro-teixeira

pedro-teixeira May 27, 2017

Owner

@rafaelpatro mais uma vez obirgado pela PR! Mas estou usando http://semver.org/ para as versões e o terceiro número está reservado para PATCHES.

Então essa versão deveria ser 4.9.0, pode alterar seu código? Precisa modificar a migração também.

This comment has been minimized.

@rafaelpatro

rafaelpatro May 27, 2017

Contributor

Beleza, já corrijo isso

rafaelpatro added a commit to rafaelpatro/correios that referenced this pull request May 27, 2017

rafaelpatro added a commit to rafaelpatro/correios that referenced this pull request May 27, 2017

@rafaelpatro

This comment has been minimized.

Contributor

rafaelpatro commented May 27, 2017

Pqp vinculei o código de issue errado, era pra ser #266.

@pedro-teixeira pedro-teixeira merged commit f13c92d into pedro-teixeira:master May 28, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@07Claudio

This comment has been minimized.

07Claudio commented Jun 30, 2017

Olá Pedro,
Atualizei o módulo para 4.9 e a correção de produtos configuráveis não foi aplicada.
Tenho aqui um produto com seis opções de escolha.
Quando a primeira opção está sem estoque o módulo mostra a informação: Este produto está sem estoque no momento. e não exibe a cotação do frete.
Poderia por gentileza me sinalizar o posso fazer para resolver.
Obrigado

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment