Save data #2
Save data #2
Conversation
…77/estrutura-basica
…cessários nesta release
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
rtakauti
requested review from
rtakauti,
carolina-y,
lyoncesar and
laerte-guimaraes
|
|
||
| trait InfoTrait | ||
| { | ||
| private $cCBrands = [ |
There was a problem hiding this comment.
Da maneira que foi implementado, se a Vindi adiciona uma nova bandeira, seria necessário realizar uma manutenção no código 😥
Você pode preencher esse array com as bandeiras obtidas através de um GET em nosso endopoint payment_methods buscando pelo code credit_card verificando se o mesmo possui o status active?
There was a problem hiding this comment.
Dei uma verificada no código e essa lista só é utilizada para traduzir os nomes dos cartões que o magento da suporte. Não necessariamente o que a api suporta. Não encontrei forma de fazer isso utilizando os dados trazidos pela api de forma que caso um novo tipo de cartão seja introduzido a tradução ocorra sem necessidade de modificar o código. Por isso será necessário manter assim :/
There was a problem hiding this comment.
Conforme conversado, estaremos liberando esse PR e criando a Issue #8 para formalizar essa questão que deve ser entregue na versão final.
Feature/trezo unit tests
Motivação
Implementado forma de pagamento no checkout, e html de informações na ordem. Já é possível criar um pedido com cartão de crédito.
Solução Proposta
Por enquanto apenas criado formulário para compras com cartão e informações no admin. Comunicação com a api e validações ficam para o próximo pull request
Este pull request contém também as modificações criadas no pull request #1 Será necessário aprovar primeiro aquele