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

Feature/ajuste no envio de informacoes #52

Merged

Conversation

Maikco
Copy link
Contributor

@Maikco Maikco commented Jul 26, 2019

Motivação

Os valores de pedidos devem ser enviados para a Vindi com todos os descontos e acréscimos aplicados, como cupons de desconto e promoções em geral.

Solução Proposta

  • Foram adicionados métodos para criar itens de desconto, taxas e frete quando esses existirem no pedido.

leonardo albuquerque added 30 commits July 19, 2018 10:23
Merge branch 'master' of github.com:vindi/vindi-magento2

# Conflicts:
#	README.md
#	app/code/Vindi/Payment/Model/Payment/Vindi.php
#	app/code/Vindi/Payment/etc/adminhtml/system.xml
#	app/code/Vindi/Payment/etc/config.xml
#	app/code/Vindi/Payment/etc/frontend/routes.xml
#	app/code/Vindi/Payment/etc/module.xml
#	app/code/Vindi/Payment/etc/payment.xml
#	app/code/Vindi/Payment/view/frontend/layout/checkout_index_index.xml
#	app/code/Vindi/Payment/view/frontend/web/template/payment/vindi.html
# Conflicts:
#	app/code/Vindi/Payment/Model/Payment/Vindi.php
#	app/code/Vindi/Payment/etc/adminhtml/system.xml
#	app/code/Vindi/Payment/etc/config.xml
#	app/code/Vindi/Payment/etc/module.xml
#	app/code/Vindi/Payment/etc/payment.xml
#	app/code/Vindi/Payment/view/frontend/layout/checkout_index_index.xml
@laerte-guimaraes laerte-guimaraes added the melhoria 🔧 New feature or request label Jul 26, 2019
@laerte-guimaraes laerte-guimaraes self-requested a review July 26, 2019 20:55
@@ -63,15 +63,15 @@ public function beforeSave()
if ($value) {
if (!$apiKey) {
throw new \Magento\Framework\Exception\LocalizedException(
sprintf(__("The api key was not set on the module basic configuration"))
__("The api key was not set on the module basic configuration")
Copy link
Contributor

Choose a reason for hiding this comment

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

🎉

@laerte-guimaraes
Copy link
Contributor

@Maikco a implementação dos testes será realizada nesse PR?

Maikco and others added 2 commits July 29, 2019 09:06
Co-Authored-By: Laerte Guimarães <[email protected]>
Co-Authored-By: Laerte Guimarães <[email protected]>
Co-Authored-By: Laerte Guimarães <[email protected]>
@laerte-guimaraes laerte-guimaraes merged commit 2beef3b into vindi:master Aug 12, 2019
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
Merge pull request vindi#52 from cedran/master
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
Merge pull request vindi#70 from cedran/master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
melhoria 🔧 New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants