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.

Sign up for GitHub

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

Jump to bottom

Adiciona compatibilidade com todos os tipos de produtos #43

Merged
merged 3 commits into from
Jan 3, 2019
Merged

Adiciona compatibilidade com todos os tipos de produtos #43

merged 3 commits into from
Jan 3, 2019

Conversation

laerte-guimaraes
Copy link
Contributor

Issue #42

Motivação

Foram relatadas falhas ao tentar realizar a compra utilizando produtos que o tipo seja diferente de 'Simples'.
Desse modo, iremos realizar uma validação para garantir a compatibilidade com os tipos de produtos existentes no Magento 2 atualmente.

Solução Proposta

  • Ajustar o comportamento para produtos to tipo Bundle, pois estavam sendo cobrados em duplicidade;
  • Tratar o código SKU dos produtos antes de enviá-los à Vindi, pois se não tratados geram erros durante a criação da Fatura.

@laerte-guimaraes laerte-guimaraes added the melhoria 🔧 New feature or request label Jan 3, 2019
@laerte-guimaraes laerte-guimaraes self-assigned this Jan 3, 2019
rtakauti
rtakauti suggested changes Jan 3, 2019
View reviewed changes
@laerte-guimaraes laerte-guimaraes merged commit a72d6f2 into master Jan 3, 2019
@laerte-guimaraes laerte-guimaraes deleted the fix/compatibilidade-com-produtos branch January 3, 2019 19:55
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
@cedran
Merge pull request vindi#43 from cedran/master
2035ef9 
fix: add lock in bill created
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
@cedran
Merge pull request vindi#25 from bizcommerce/master
476bd9b 
Merge pull request vindi#43 from cedran/master
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
@cedran
Merge pull request vindi#43 from bizcommerce/master
cc5983d 
Merge pull request vindi#61 from cedran/master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vitorkusiaki vitorkusiaki vitorkusiaki approved these changes

@rtakauti rtakauti rtakauti approved these changes

Assignees

@laerte-guimaraes laerte-guimaraes

Labels
melhoria 🔧 New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@laerte-guimaraes @vitorkusiaki @rtakauti