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

Adiciona método de pagamento Pix #166

Merged
merged 74 commits into from
Apr 4, 2024

Conversation

devaguia
Copy link
Contributor

@devaguia devaguia commented Mar 7, 2024

O que mudou

  • Criada uma nova classe PHP para o novo gateway de pagamento;
  • Adicionado funções de processamento de pagamento com o gateway de pagamento PIX;
  • Adicionado visualização de QR Code e linha digitável de PIX na página de agradecimento;
  • Criada a renovação do QR Code PIX;
  • Melhorando compatibilidade com HPOS;
  • Refatorando função bill_paid do arquivo de Webhook por apontamento do codeclimate;

Motivação

Permitir que os usuários possam efetuar compras utilizando o método de pagamento PIX.

Solução proposta

Descrito no tópico 'O que mudou'.

Como testar

Configurar o método de pagamento PIX no painel administrativo e efetuar compras de produtos utilizando o novo método.

@devaguia devaguia requested a review from a team as a code owner March 7, 2024 10:53
@devaguia
Copy link
Contributor Author

@KarinaSakata Sobre o seu comentário:

❗ OBS: Tentamos realizar a renovação de uma assinatura através da app de VR, porém, neste cenário, apesar do pedido ser criado no painel da página Woocomerce como pagamento pendente, não conseguimos visualizar o novo QR CODE.
Ficamos em dúvida se este ponto deveria fazer parte deste PR ou se será tratado posteriormente.

De fato o plugin não exibe o QR Code ou boleto para as renovações, somente no primeiro ciclo após a compra.

@rai-gondim-vindi
Copy link

@KarinaSakata Sobre o seu comentário:

❗ OBS: Tentamos realizar a renovação de uma assinatura através da app de VR, porém, neste cenário, apesar do pedido ser criado no painel da página Woocomerce como pagamento pendente, não conseguimos visualizar o novo QR CODE.
Ficamos em dúvida se este ponto deveria fazer parte deste PR ou se será tratado posteriormente.

De fato o plugin não exibe o QR Code ou boleto para as renovações, somente no primeiro ciclo após a compra.

@devaguia esse é um comportamento esperado ou é um comportamento que será corrigido nesse PR?

@devaguia
Copy link
Contributor Author

@KarinaSakata Sobre o seu comentário:

❗ OBS: Tentamos realizar a renovação de uma assinatura através da app de VR, porém, neste cenário, apesar do pedido ser criado no painel da página Woocomerce como pagamento pendente, não conseguimos visualizar o novo QR CODE.
Ficamos em dúvida se este ponto deveria fazer parte deste PR ou se será tratado posteriormente.

De fato o plugin não exibe o QR Code ou boleto para as renovações, somente no primeiro ciclo após a compra.

@devaguia esse é um comportamento esperado ou é um comportamento que será corrigido nesse PR?

É o esperado @KarinaSakata @raicgvindi

@rai-gondim-vindi
Copy link

Foi gerado um pix com vencimento de 24 horas ontem 17:16
Hoje, após 14:16, na tela do pedido #755 aparecia que o qr code do pix estava expirado.

antes de 14:16
Screenshot from 2024-03-26 14-15-09

após 14:16
Screenshot from 2024-03-26 14-22-21

MAX_DAYS_TO_KEEP_WAITING_PAYMENT
Screenshot from 2024-03-26 14-48-22

@devaguia
Copy link
Contributor Author

devaguia commented Mar 26, 2024

Foi gerado um pix com vencimento de 24 horas ontem 17:16 Hoje, após 14:16, na tela do pedido #755 aparecia que o qr code do pix estava expirado.

@raicgvindi Vou verificar. Pode ser alguma configuração de fuso, pois o plugin compara a data salva no banco com a data atual na renderização da página.

@devaguia
Copy link
Contributor Author

É isso mesmo @raicgvindi, deveria pegar 16:04. Já mando uma correção.

Captura de Tela 2024-03-26 às 4 07 13 PM

Copy link

codeclimate bot commented Mar 26, 2024

Code Climate has analyzed commit 4875aee and detected 0 issues on this pull request.

View more on Code Climate.

@devaguia
Copy link
Contributor Author

Foi gerado um pix com vencimento de 24 horas ontem 17:16 Hoje, após 14:16, na tela do pedido #755 aparecia que o qr code do pix estava expirado.

@raicgvindi Enviado para o ambiente de teste. Pode verificar?

Copy link

@rai-gondim-vindi rai-gondim-vindi left a comment

Choose a reason for hiding this comment

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

@devaguia fiz só mais 2 comentários relacionados à indentação e nada mais.
Reparei também que o code climate ainda apresentou alguns problemas nos arquivos src/routes/RoutesApi.php e src/services/Webhooks.php

Retirando isso, por mim já está 100%.
Depois que você ver o que precisa ajeitar, comenta aqui que dou o approve.

src/VindiWoocommerce.php Show resolved Hide resolved
src/utils/PaymentGateway.php Show resolved Hide resolved
@rai-gondim-vindi rai-gondim-vindi self-requested a review April 2, 2024 13:14
@thais-kusuki-vindi thais-kusuki-vindi changed the title Build/add_pix_gateway Adiciona método de pagamento Pix Apr 3, 2024
@thiago-firmino-vindi thiago-firmino-vindi merged commit b552cfc into vindi:master Apr 4, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants