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

Insere necessidade de setup do módulo #40

Merged
merged 13 commits into from
Sep 25, 2018
Merged

Conversation

laerte-guimaraes
Copy link
Contributor

Motivação

O PR #39 foi realizado para aprovação no marketplace, porém identificamos a necessidade de exigir um setup do módulo, pois o status default do método de pagamento Vindi, era habilitado, porém por não existir a chave API, eram retornados erros ao consumidor final.

Solução Proposta

  • Ajustar e finalizar verificação de status do módulo;
  • Setar o status default do método cartão de crédito para desabilitado

@laerte-guimaraes laerte-guimaraes self-assigned this Sep 19, 2018
@laerte-guimaraes
Copy link
Contributor Author

@FlaviaFortes padronizei todos os arquivos alterados para aspas simples, acredito ser mais performático 😄

@laerte-guimaraes laerte-guimaraes removed the request for review from lyoncesar September 25, 2018 13:35
@laerte-guimaraes laerte-guimaraes merged commit 9d896d6 into master Sep 25, 2018
@laerte-guimaraes laerte-guimaraes deleted the valida-mktplace branch September 25, 2018 14:07
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
Merge pull request vindi#21 from bizcommerce/master
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
Merge pull request vindi#40 from cedran/master
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
Merge pull request vindi#58 from cedran/master
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
Merge pull request vindi#40 from bizcommerce/master
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.

3 participants