Skip to content

refactor: createOrderAtPagarme returns object #387

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

Merged
merged 2 commits into from
Jun 25, 2025

Conversation

RafaMelazzo
Copy link
Contributor

@RafaMelazzo RafaMelazzo commented Jun 24, 2025

Paying

Important

Certifique-se de criar o PR para a branch stg.

Tarefa ECPJ-54

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

Método createOrderAtPagarme foi modificado no Core e agora retorna um objeto, no lugar de um array contendo apenas um objeto. Essa função é utilizada no Magento uma única vez. Este PR refatora para tratar a resposta como objeto.

Testes

Testado fluxos de compras com Cartão de Crédito, Pix e Boleto

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.

@RafaMelazzo RafaMelazzo requested review from Copilot and a team June 24, 2025 21:36
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the handling of the createOrderAtPagarme response to match its new return type (an object instead of a single-element array).

  • Removed use of current() when assigning the Pagar.me order result.
  • Updated $pagarmeOrder assignment to receive an object directly.

@RafaMelazzo RafaMelazzo merged commit fda95e6 into stg Jun 25, 2025
3 of 4 checks passed
@RafaMelazzo RafaMelazzo deleted the feat/ECPJ-54-allow-18x-installments branch June 25, 2025 15:38
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.

4 participants