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

fix: use the appropriate getters (woocommerce CRUD) #527

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

gutobenn
Copy link
Contributor

@gutobenn gutobenn commented Jan 3, 2025

Qual o tipo de PR é esse? (marque todos os aplicáveis)

  • Refatoração
  • Adição de funcionalidade
  • Correção de bug
  • Otimização
  • Atualização de documentação

Descrição

Este PR corrige warnings que estavam sendo gerados. A maioria deles por não utilizar os getters CRUD do woocommerce.
Issue relacionada: #480

Atualização:: Agora vi que há outro PR recente (#523) que corrige uma parte dos problemas que meu PR corrige.

Cenários testados

Compra no cartão de crédito, em loja com HPOS ativado

Copy link

⚠️ Este repositório ainda não está catalogado no Backstage. ⚠️

Por favor, catalogue-o seguindo as instruções nesta documentação. [Via VPN].

💁 Qualquer problema ou dúvida, estamos no Slack, basta abrir um ticket no canal #help-foundation-platform.

@gutobenn gutobenn changed the title Fix: use the appropriate getters (woocommerce CRUD) fix: use the appropriate getters (woocommerce CRUD) Jan 3, 2025
@fabiano-mallmann fabiano-mallmann merged commit 92172a1 into pagarme:develop Jan 7, 2025
2 of 3 checks passed
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.

3 participants