Skip to content

Feature: Update GoCardless plugin to support Instant Bank Pay (IBP) feature#21

Open
cdave26 wants to merge 3 commits into
killbill:mainfrom
cdave26:feature/instant-bank-pay
Open

Feature: Update GoCardless plugin to support Instant Bank Pay (IBP) feature#21
cdave26 wants to merge 3 commits into
killbill:mainfrom
cdave26:feature/instant-bank-pay

Conversation

@cdave26
Copy link
Copy Markdown

@cdave26 cdave26 commented Feb 18, 2026

  • Updated gocardless-pro dependency version to 7.6.0.
  • Added new servlets for handling IBP checkout and completion flows.
  • Implemented logic for creating and fulfilling Billing Requests for instant payments.
  • Enhanced README with details on IBP payment flow and usage examples.

- Updated gocardless-pro dependency version to 7.6.0.
- Added new servlets for handling IBP checkout and completion flows.
- Implemented logic for creating and fulfilling Billing Requests for instant payments.
- Enhanced README with details on IBP payment flow and usage examples.
Copy link
Copy Markdown
Contributor

@reshmabidikar reshmabidikar left a comment

Choose a reason for hiding this comment

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

There are ci failures due to compilation errors. Could you please address those?

- Updated gocardless-pro dependency version to 7.6.0.
- Added new servlets for handling IBP checkout and completion flows.
- Implemented logic for creating and fulfilling Billing Requests for instant payments.
- Enhanced README with details on IBP payment flow and usage examples.
- fix: remove error details from PaymentPluginApiException
- Consolidated error response creation for missing parameters and invalid currency in GoCardlessInstantCheckoutServlet and GoCardlessInstantCompleteServlet.
- Simplified metadata retrieval in GoCardlessPaymentPluginApi, removing redundant null checks.
- Improved exception messages for better clarity in error handling.
Copy link
Copy Markdown
Contributor

@reshmabidikar reshmabidikar left a comment

Choose a reason for hiding this comment

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

There are still compilation errors. Could you please address those?

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