claudiosanches / woocommerce-extra-checkout-fields-for-brazil

Brazilian Market on WooCommerce - WordPress Plugin.
http://wordpress.org/plugins/woocommerce-extra-checkout-fields-for-brazil/
93 stars 59 forks source link

better settings for cell_phone field #217

Closed luizbills closed 1 year ago

luizbills commented 2 years ago

216

luizbills commented 2 years ago

@claudiosanches usei a função wc_get_var e acho que ela não existia antes da versão 3.0 do woocommerce. Acha necessário criar um polyfill dela? Se o polyfill for necessário, em qual arquivo seria melhor para colocá-lo?

claudiosanches commented 1 year ago

@luizbills esta tudo bem usar a função nova. Você pode atualizar o Pull Request para resolver os conflitos?

luizbills commented 1 year ago

@claudiosanches assim que der resolvo os conflitos.

luizbills commented 1 year ago

@claudiosanches resolvido os conflitos e testado.


Vi que agora tem valores padrão (não lembro se antigamente tinha). Talvez não fosse legal colocar como padrão uma dessas opções abaixo?

  1. Deixar campo "celular" desativado.
  2. Deixar o campo telefone com label "Celular".

Cito isso, pois hoje em dia no Brasil quase ninguém usa telefone fixo. Enfim, só uma sugestão.

claudiosanches commented 1 year ago

@luizbills Podemos trocar o padrão sem problema, vai a opção escolhida em novas instalações, então é tranquilo para quem já tem instalado e configurado.

luizbills commented 1 year ago

@luizbills Podemos trocar o padrão sem problema, vai a opção escolhida em novas instalações, então é tranquilo para quem já tem instalado e configurado.

Qual daquelas sugestão de padrão vc acha melhor? Pessoalmente, eu gosto mais do número 2.

claudiosanches commented 1 year ago

Eu também prefiro a número 2, faz mais sentido atualmente.

luizbills commented 1 year ago

Ta feito, então.

claudiosanches commented 1 year ago

@luizbills muito obrigado, até final de semana já reviso e testo certinho pra fazer deploy. Valeu!

claudiosanches commented 1 year ago

Muito obrigado!