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

Variations #7

Merged
merged 7 commits into from
May 5, 2016
Merged

Variations #7

merged 7 commits into from
May 5, 2016

Conversation

asagalo
Copy link
Contributor

@asagalo asagalo commented Apr 27, 2016

Esse PR adiciona o suporte a produtos e assinatura variáveis, ele faz isso utilizando os próprios tipos de variações do WC. Na Vindi os itens da bill serão exibidos as variações do seguinte formato "Nome do produto (Variação)", caso haja mais de uma opção na variação do produto elas irão separadas pelo caractere (-). Ex: Nome do Produto (Variação 1 - Variação 2). Também foi adicionado uma correção para a issue #6, que corrige o problema da exibição de plano anuais.

@lyoncesar
Copy link
Contributor

Encontrei um problema quando a variação possui uma variação que não foi especificada.
Ex:
camiseta azul(1ª variação), tamanho M(2ª variação).

A primeira variação foi especificada nos atributos do produto, mas não foi adicionada nas variações.
Quando crio uma compra desse produto a segunda variação não é enviada para a Vindi.

@lyoncesar
Copy link
Contributor

Na área do cliente está aparecendo a opção de upgrade/downgrade de plano.
Acho que o melhor será lançar sem essa opção e depois lançamos uma outra atualização.

@asagalo
Copy link
Contributor Author

asagalo commented May 5, 2016

Essa feature de Upgrade e Downgrade é configurável no próprio WSC. Eu adicionei um aviso indicando a desativação da opção. E alertando que podem ocorrer divergências nos preços das assinaturas caso a opção não seja desabilitada.

@lyoncesar lyoncesar merged commit 0bf866b into master May 5, 2016
@lyoncesar lyoncesar deleted the feature/variations branch May 5, 2016 17:03
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.

2 participants