-
Notifications
You must be signed in to change notification settings - Fork 12
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 da alteração do método de pagamento #184
Open
lucastgama
wants to merge
237
commits into
vindi:master
Choose a base branch
from
lucastgama:apiki_alteracao_metodo_pagamento
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Correção da alteração do método de pagamento #184
lucastgama
wants to merge
237
commits into
vindi:master
from
lucastgama:apiki_alteracao_metodo_pagamento
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…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.
…alizar estado da assinatura
…ment e na rota updateSubscription
…reensão do código
Code Climate has analyzed commit 47d9d1f and detected 0 issues on this pull request. View more on Code Climate. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
O que mudou
Anteriormente, ao tentar alterar o método de pagamento de uma assinatura, o sistema tratava a ação como uma nova compra, resultando no erro "order id em uso". Agora, o sistema identifica corretamente quando a solicitação é para uma assinatura existente, permitindo a alteração do método de pagamento sem erros. Além disso, foi adicionada a compatibilidade com cartões Mastercard que possuem numeração iniciada pelo dígito 2.
Motivação
Resolver o problema que impedia a alteração do método de pagamento em assinaturas existentes, proporcionando uma experiência mais fluida para o usuário. Garantir a aceitação de cartões Mastercard com novas faixas numéricas.
Solução proposta
Como testar
2223003122003222
.