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

Versão 2.0 - Novo Motor de Recorrência #118

Merged
merged 467 commits into from
Aug 30, 2024

Conversation

contardi
Copy link
Contributor

@contardi contardi commented Jul 30, 2024

O que mudou

Foi criada uma nova versão, com alteração "major" dentro do versionamento, para que o novo motor de recorrência possa ser utilizado de maneira diferente, com os planos sendo criados a parte podendo ser relacionados com um ou ais produtos.

Motivação

Foi levantada a necessidade do novo motor de recorrência por conta de limitações do atual modelo de recorrência, possibilitando a venda de diferentes tipos de produtos como recorrência.

Solução Proposta

Foi refatorado o código e criado a possibilidade de criar planos separadamente, dentro da edição do produto você seleciona os planos que esse produto faz parte e na escolha do produto os planos aparecem como opções na hora da compra
A ideia foi tirada de outras lojas com assinaturas que possuem modelo similar e assim, criar um modelo mais familiar com o que é visto no mercado

Como testar

Foi criado um plano de testes com o link da loja utilizada, esses documentos foram enviados diretamente à equipe da Vindi.

Observações

Será preciso criar a tag v2.0.0 para essa versão

cedran and others added 30 commits June 14, 2024 10:59
Merge pull request #48 from cedran/master
fix: webhook bill_created
Merge pull request #49 from cedran/master
fix: webhook bill_created
Merge pull request #50 from cedran/master
fix: webhook bill_created
Merge pull request #51 from cedran/master
fix: webhook bill_created
Merge pull request #52 from cedran/master
Merge pull request #53 from cedran/master
Merge pull request #35 from bizcommerce/master
Merge pull request #54 from cedran/master
Merge pull request #55 from cedran/master
Merge pull request #56 from cedran/master
Merge pull request #38 from bizcommerce/master
@cedran
Copy link
Contributor

cedran commented Aug 19, 2024

LOGS: Notamos que alguns dados sensíveis estão visíveis nos logs. Se possível, sugerimos utilizar uma máscara (como já está sendo feito com o e-mail e número do cartão do customer).

1 - CVV visível image

2- Documento visível image

Foram aplicadas as devidas máscaras ao CVV e ao documento.

@cedran
Copy link
Contributor

cedran commented Aug 20, 2024

  1. Realizei um ajuste para corrigir a duplicação de mensagens.

Quando tentamos realizar a compra de uma assinatura, com um produto avulso já dentro do carrinho, por algum motivo, às vezes a mensagem de erro não aparece para o cliente. mensagem-erro mensagem-erro

Eu não consegui simular esse erro. Imagino que pode ter relação com o sistema de cacheamento no Magento.
Contudo, implementei um ajuste que tenta garantir que a chamada da mensagem será executada.

@cedran
Copy link
Contributor

cedran commented Aug 20, 2024

GESTÃO DE ASSINATURAS:

Dúvida: Ao removermos o desconto na Vindi e realizar a renovação da assinatura, o desconto permanece sendo aplicado no Magento, mesmo que tenha sido removido antes da renovação. Este comportamento é mesmo esperado?

image image image

O fluxo de atualização de assinaturas realizadas diretamente no portal da Vindi para a loja Magento está no backlog e será implementado em uma nova feature.

'email' => $customer->getEmail(),
'registry_code' => $registryCode,
'code' => $uniqueCode,
'phones' => $this->formatPhone($billingAddress->getTelephone()),
Copy link
Contributor

Choose a reason for hiding this comment

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

Notei que a função formatPhone está retornando uma string, com landline para telefones fixos e mobile para celular. porém o parâmetro phones na API espera receber um array de objetos com os parâmetros phone_type, number e extension(campo opcional caso o número tenha ramal).

Segue um exemplo de requisição com o envio de um número de telefone:
POST v1/customers

{
    "name": "John Doe",
    "phones": [
        {
            "phone_type": "mobile",
            "number": "5511912341234"
        }
    ]
}

Link para documentação da API

Analisando o histórico o problema não foi implementado neste PR, mas seria bom deixarmos mapeado para esta ou uma próxima entrega.

Copy link

@leydiane-matias-vindi leydiane-matias-vindi Aug 20, 2024

Choose a reason for hiding this comment

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

Notei que a função formatPhone está retornando uma string, com landline para telefones fixos e mobile para celular. porém o parâmetro phones na API espera receber um array de objetos com os parâmetros phone_type, number e extension(campo opcional caso o número tenha ramal).

Segue um exemplo de requisição com o envio de um número de telefone: POST v1/customers

{
    "name": "John Doe",
    "phones": [
        {
            "phone_type": "mobile",
            "number": "5511912341234"
        }
    ]
}

Link para documentação da API

Analisando o histórico o problema não foi implementado neste PR, mas seria bom deixarmos mapeado para esta ou uma próxima entrega.

OI Leo,
De fato nao implementamos esse item nesse PR, mas já anotei para o backlog do Magento. Nas próximas entregas, colocamos esse ponto. Obrigada por sinalizar.

Copy link
Member

Choose a reason for hiding this comment

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

Issue aberta do bug: #82

@karina-sakata-vindi
Copy link

CHECKOUT:
Tentando validar o critério de aceite Está pronto quando a cada renovação de ciclo, o Boleto for atualizado, notamos que a renovação quando o método de pagamento é boleto, não está retornando o link do boleto atualizado, nem na conta do cliente no checkout, nem no painel admin.

image
image

Helper/Api.php Outdated Show resolved Hide resolved
@cedran
Copy link
Contributor

cedran commented Aug 22, 2024

CHECKOUT: Tentando validar o critério de aceite Está pronto quando a cada renovação de ciclo, o Boleto for atualizado, notamos que a renovação quando o método de pagamento é boleto, não está retornando o link do boleto atualizado, nem na conta do cliente no checkout, nem no painel admin.

image image

A url do boleto não estava sendo salva adequadamente no pedido na renovação da assinatura.
Implementei uma correção para que ele guarde as informações de forma correta.

@thais-kusuki-vindi thais-kusuki-vindi merged commit ad0e4ad into vindi:development Aug 30, 2024
carolineesteves pushed a commit to bizcommerce/vindi-magento2 that referenced this pull request Sep 13, 2024
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.

10 participants