-
Notifications
You must be signed in to change notification settings - Fork 5
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/add-pix-payment-method-pagarme-V3 #379
Conversation
3a047ef
to
746da15
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nesse PR aqui os testes passaram: #371
Então acredito que é só corrigir o erro que está dando :)
b6f1daa
to
800083c
Compare
Removed line 109 "assertEquals($this->anticipation->getPaymentDate(), $this->expectedPaymentDate);" from "PagarMe\Acceptance\BulkAnticipationContext::mustAnticipationContainSameData()" because UTC makes test fail.
800083c
to
fb2c8d4
Compare
Changed refund amount because new rule of refunds make impossible old refund value
976eb6f
to
9fbef7a
Compare
f1e2e09
to
82a4cd6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boa, aprovado! Acho que só é bacana juntar os commits que mexeram em alguns arquivos e depois voltaram ao que era antes, pra evitar alguma confusão futura. E também alterar a descrição do PR, a parte que diz que a versão não tem testes automatizados, pois agora eles já estão passando com sucesso :)
Obs.: Entendo que tudo bem ainda não ter testes específicos de PIX, pois no momento precisa de companies e configurações específicas pra poder testar.
Olá amigos. Espero que eu não esteja escrevendo e fazendo perguntas nos locais errados, mas preciso perguntar. Existe alguma previsão de quando essa PR estará no master? Existe outra PR de PIX? Essa PR entrará na versão 3 ainda ou só entrará na 4, saberiam dizer? Obrigado. |
Oi @rstriquer, tudo bem? A previsão de deploy da v3 do nosso SDK é 15/12 terça-feira. Para a v4 não precisou de alteração no SDK, devido a forma que ele foi construído para aceitar os parâmetros através de um array, então basta enviar os novos campos referentes ao PIX. Exemplo:
Atualmente para ativar o PIX aqui na Pagar.me, mesmo que em ambiente de testes, você pode enviar um e-mail para [email protected], ok? |
@willian-soaresferreira a versão marcada no home do projeto é a v3.8.2, entendo que ela é a stable, mas eu posso utilizar, p. ex., a v4.1.2 em produção, ela está marcada como stable também? Grato. |
Oi @rstriquer! A versão mais atual e que sugerimos o uso para novos projetos é a v4, e o build mais recente é de fato o v.4.1.2. Inclusive a branch "main" do projeto pagarme-php é a v4. Ainda damos manutenção na v3 no momento pois tem clientes com projetos já existentes que ainda a utilizam. Qualquer dúvida por favor me avise :) |
Olá pessoal, |
O retorno da API é o parâmetro "pix_qr_code" que pode ser utilizado no aplicativo de banco do comprador ou renderizado na forma de QR Code visual. Essa etapa de renderizar já é por conta da aplicação da loja. |
Obrigado @Matheus-Maciel |
Descrição
O PR visa adicionar o método de pagamento PIX no SDK de PHP V3.
Número da Issue
Não existe
Testes Realizados
Por hora foram realizados apenas testes manuais visto que os testes do SDK precisam ser refatorados