vindi / vindi-woocommerce

Plugin Vindi para pagamento recorrente no Woocommerce
https://vindi.com.br
GNU General Public License v3.0
7 stars 12 forks source link

fix: invertendo verificação que dispara a notificação para ativar a o… #120

Closed devaguia closed 2 years ago

devaguia commented 2 years ago

117

O que mudou

Verificação responsável por disparar a notificação para ativar a opção "Turn off Automatic Payments".

Motivação

Descreva a motivação deste PR. Explique qual é o problema que queremos resolver.

Solução proposta

Inverter a verificação:

/**
* Warning if WCS automatic payments settings are disabled
**/
public function wcs_automatic_payment_settings()
{
    if('yes' !== get_option('woocommerce_subscriptions_turn_off_automatic_payments')) {
      $this->get_template('wcs-automatic-payment-deactivated-message.html.php');
    }

    return;
}

/vindi-payment-gateway/src/includes/admin/Settings.php line 334~343

Como testar

Ativar a opção nas configurações do plugin WooCommerce Subscription.

codeclimate[bot] commented 2 years ago

Code Climate has analyzed commit 1a85c183 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Style 2

View more on Code Climate.

thalita-nascimento-vindi commented 2 years ago

Boa Tarde @aguiarrdev tudo bem?

poderia nos enviar um commit vazio para destravar o codeclimate? git commit --allow-empty -m "Empty-Commit"

Aguardo.

thalita-nascimento-vindi commented 2 years ago

@aguiarrdev Boa Tarde. Poderia validar esse também?

Aguardamos.

codeclimate[bot] commented 2 years ago

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

View more on Code Climate.