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

Correção de ID de Usuário Vindi e Melhoria no Cancelamento de Faturas #183

Merged

Conversation

lucastgama
Copy link
Contributor

O que mudou

Este PR implementa melhorias e correções de bugs no plugin, visando resolver problemas relacionados ao ID de usuário do sistema Vindi e ao processamento de faturas no caso de cancelamentos. Agora, o ID do usuário Vindi é validado e armazenado corretamente no banco de dados do WordPress, evitando erros de ID duplicado. Além disso, foi corrigido o problema de faturas pendentes que continuavam sendo geradas após o cancelamento de pedidos no WooCommerce.

Motivação

O sistema da Vindi, ao processar compras, exige que o ID do usuário seja informado. Quando um usuário não possui esse ID, é feita uma requisição para criá-lo no sistema da Vindi. No entanto, havia um problema em que o ID gerado desaparecia do banco de dados do WordPress, resultando em erros ao tentar criar o usuário na Vindi, pois o ID do usuário WP já estava em uso. Além disso, quando um pedido era cancelado no WooCommerce, o sistema da Vindi ainda gerava faturas devido à falta de comunicação sobre o cancelamento.

Solução proposta

Correção do ID de Usuário Vindi: Foi implementada uma nova rota de API que verifica se o usuário já existe no sistema Vindi, usando o ID do WordPress. Caso o usuário exista, o ID da Vindi é recuperado e armazenado corretamente no banco de dados do WordPress. Caso contrário, o fluxo de criação do usuário continua normalmente.

Correção no Cancelamento de Pedidos: Foi feita uma melhoria no disparo da comunicação de cancelamento entre o WooCommerce e o sistema da Vindi. Agora, quando um pedido é cancelado no WooCommerce, o sistema da Vindi recebe a informação de cancelamento e não gera mais faturas para esse pedido.

Como testar

  1. Testar Criação de Usuário na Vindi
    1.1 Efetue uma compra no sistema WooCommerce com um usuário que ainda não exista no sistema Vindi.
    1.2 Após a compra, vá até o painel do sistema da Vindi e verifique se o usuário foi criado corretamente, validando se o ID da Vindi foi atribuído.
    1.3 Acesse o banco de dados do WordPress e procure pelo cliente.
    Verifique se o Vindi ID foi armazenado corretamente na tabela de clientes no banco de dados do WP.
    1.4 Realize uma nova compra com o mesmo usuário que foi criado anteriormente.
    1.5 Verifique se não ocorre erro relacionado ao Vindi ID (como antes). O esperado é que o sistema reconheça o ID já existente e não gere erros.
    1.6 Acesse o banco de dados do WordPress e apague o Vindi ID do cliente.
    1.7 Simule uma nova compra com o mesmo usuário.
    1.8 O sistema deverá identificar que o usuário já existe no contexto da Vindi e enviar o ID da Vindi novamente para ser salvo no banco de dados do WP, evitando o erro de Vindi ID null.

  2. Testar Cancelamento de Pedido e Fatura
    2.1 Simule uma compra de assinatura no WooCommerce.
    2.2 Vá até a seção "My Orders" no WooCommerce e solicite o cancelamento do pedido.
    2.3 Verificar Cancelamento no WooCommerce e Vindi: O pedido deverá ser marcado como "Cancelamento Pendente" no WooCommerce e como "Cancelado" no sistema da Vindi, sem gerar novas faturas.

…com as novas versões do PHP 8.3 e do WC 8.5.2
…remainder na condiçao quando o resto menor ou igual ao preço total, antes tinha um else e esse else foi removido para que o code climate pare com a issue
…era remainder - fullprice agora é apenas fullprice
… duplicação do desconto e na aplicação incorreta do desconto percentual em produtos simples.
@lucastgama lucastgama requested a review from a team as a code owner November 14, 2024 20:10
@lucastgama lucastgama changed the title Apiki subscription pending cancel Correção de ID de Usuário Vindi e Melhoria no Cancelamento de Faturas Nov 14, 2024
lucastgama and others added 23 commits November 18, 2024 08:54
Adicionando 1.3.5
Copy link

codeclimate bot commented Nov 28, 2024

Code Climate has analyzed commit 6fdc646 and detected 0 issues on this pull request.

View more on Code Climate.

@leticia-santos-vindi leticia-santos-vindi merged commit a7bfe5d into vindi:master Nov 28, 2024
1 of 2 checks passed
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.

2 participants