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

Verificar chave API na habilitação dos métodos de pagamento #15

Closed
laerte-guimaraes opened this issue Aug 8, 2018 · 3 comments
Closed

Verificar chave API na habilitação dos métodos de pagamento #15

laerte-guimaraes opened this issue Aug 8, 2018 · 3 comments
Labels
melhoria 🔧 New feature or request necessita atenção 🚨 Extra attention is needed

Comments

@laerte-guimaraes
Copy link
Contributor

É possível realizar a verificação da Chave API ao tentar habilitar um método de pagamento da Vindi?

O objetivo, é não deixar o cliente habilitar os métodos (salvar), caso a chave API não esteja setada.
Desse modo, não é necessário validar $this->apiKey no request da API, pois caso não possua chave API, não irá chegar no request.
@leonardola
Copy link
Contributor

corrigido no pull request #13 no arquivo app/code/Vindi/Payment/Model/Config/Backend/ApiKeyValidator.php

@laerte-guimaraes
Copy link
Contributor Author

@leonardola é possível separar em um novo PR?
Acredito que para deixar a validação mais precisa, você pode utilizar o endpoint
GET /v1/merchants/current e verificar se o status é diferente de inactive

@laerte-guimaraes laerte-guimaraes added melhoria 🔧 New feature or request necessita atenção 🚨 Extra attention is needed labels Aug 10, 2018
leonardola pushed a commit to trezoteam/vindi-magento2 that referenced this issue Aug 10, 2018
adicionado validação da chave da api fixes vindi#15
5ef45e7
@leonardola leonardola mentioned this issue Aug 10, 2018
Merged
@leonardola
Copy link
Contributor

criado pull request #21 com estas mudanças

carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this issue Sep 13, 2024
@cedran
Merge pull request vindi#15 from bizcommerce/master
c299289 
Merge pull request vindi#33 from cedran/master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
melhoria 🔧 New feature or request necessita atenção 🚨 Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@leonardola @laerte-guimaraes