Closed luizdalcicoapisys closed 3 years ago
leo-araujo
commented
3 years ago @luizdalcicoapisys @laerte-guimaraes Realizei alguns testes de compras e tive alguns problemas. segue:
Link do boleto na página de conclusão de compra gera erro:
Todos os webhooks enviados estão recebendo 302:
Pesquisei pra ver se conseguimos realizar alguma configuração para contornar isto mas sem sucesso.
laerte-guimaraes
commented
3 years ago Realizei alguns testes de compras e tive alguns problemas. segue Link do boleto na página de conclusão de compra gera erro
@LeoAraujo01 poderia apagar resquícios de outras versões do plugin e testar novamente com essa branch?
Lembrando que esse tratamento é apenas em permitir a instalação do plugin em multisite.
Um bom ponto é realizar o teste de fluxo de compra:
Mas não são impeditivos para um PR desse tipo, e devem ser abertos em novas Issues tudo bem? 😉
codeclimate[bot]
commented
3 years ago Code Climate has analyzed commit 4727a81e and detected 0 issues on this pull request.
View more on Code Climate.
O que mudou
Agora a verificação de dependências necessárias para o correto funcionamento do vindi-woocommerce está funcionando tanto para o Wordpress em sua instalação comum como no ambiente Wordpress Multisite.
Motivação
Ao tentar instalar o plugin vindi-woocommerce no Wordpress Multisite, a página do admin acusava ausência do plugin Woocommerce >= 3.0, mesmo ele estando instalado. Para mais detalhes, ver issue #51 .
Solução proposta
Foi feita uma refatoração da classe que checa as dependências do plugin. Ao invés de buscar informações do plugin pelo diretório, a checagem agora leva em conta as informações salvas na tabela de opções do Wordpress. O código também ficou mais simples e mais inteligível com a mudança.
Como testar
Instalar o plugin da Vindi em ambiente normal e Multisite, levando em conta os três plugins que são dependências obrigatórias:
Closes #51