Skip to content

fix frame payment renderer component#16

Open
PanMisza wants to merge 1 commit into
bluepayment-plugin:masterfrom
PanMisza:fix-frame-renderer-component
Open

fix frame payment renderer component#16
PanMisza wants to merge 1 commit into
bluepayment-plugin:masterfrom
PanMisza:fix-frame-renderer-component

Conversation

@PanMisza
Copy link
Copy Markdown

Magento w pewnych sytuacjach wymaga aby payment renderer miało metody, których bluepayment-prepend i bluepayment-append nie mają.

Przykładowy scenariusz, zakładający że żadna z metod płatności bluepayment nie jest dostępna dla metod dostawy za pobraniem:

  1. Uzupełnienie adresu dostawy oraz wybór metody dostawy (innej niż za pobraniem).
  2. Przejście do następnego kroku.
  3. Wybór jednej z metod płatności bluepayment.
  4. Powrót do poprzedniego kroku.
  5. Zmiana metody dostawy na jakąkolwiek "za pobraniem".
  6. Przejście do następnego kroku (tutaj błąd).

Użytkownik nie może przejść dalej, a w konsoli zobaczymy błąd z magento/module-checkout/view/frontend/web/js/view/payment/list.js [removeRenderer] - wykonywane jest tam na naszych rendererach disposeSubscriptions(), a takiej metody nie mamy.

Zmieniłem komponent tych rendererów z uiComponent na Magento_Checkout/js/view/payment/default i to rozwiązuje ten problem.

Copilot AI review requested due to automatic review settings August 20, 2025 08:36
Copy link
Copy Markdown

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

Fixes a payment renderer component issue in Magento's checkout process by changing the base component from uiComponent to Magento_Checkout/js/view/payment/default. This resolves an error that occurs when switching delivery methods after selecting a bluepayment payment method.

  • Change component base class for bluepayment-prepend and bluepayment-append renderers
  • Ensures proper lifecycle method availability (disposeSubscriptions) required by Magento's payment list component

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread view/frontend/web/js/view/payment/bluepayment.js
@PanMisza PanMisza changed the title fix frame payment renderer omponent fix frame payment renderer component Aug 20, 2025
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.

2 participants