Skip to content

docs(refunds): clarify conditional IBAN/BIC requirement#77

Merged
jfrosorio merged 1 commit into
masterfrom
docs-76
Jul 17, 2026
Merged

docs(refunds): clarify conditional IBAN/BIC requirement#77
jfrosorio merged 1 commit into
masterfrom
docs-76

Conversation

@jfrosorio

Copy link
Copy Markdown
Contributor

Closes #76.

Live testing on the sandbox (2026-07-17) confirmed that the refund iban and bic fields, while marked optional in Eupago's API docs, are conditionally required: methods without a direct refund path (e.g. Multibanco) reject with IBAN_MISSING unless an IBAN is given, and an IBAN without its matching BIC is rejected with BIC_INVALID — Eupago does not derive the BIC. Direct-path methods such as MB WAY need neither.

This updates docs/refunds.md to spell that out, including the rejection codes users will see.

@jfrosorio
jfrosorio requested a review from Copilot July 17, 2026 16:06
@jfrosorio jfrosorio self-assigned this Jul 17, 2026
@jfrosorio jfrosorio added the documentation Improvements or additions to documentation label Jul 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the refunds documentation to reflect Eupago’s observed “conditionally required” behavior for iban/bic during refunds, reducing failed refund attempts caused by relying on the upstream API docs’ “optional” labeling.

Changes:

  • Clarifies that direct-path refund methods (e.g. MB WAY) require no iban/bic.
  • Documents that non-direct-path methods (e.g. Multibanco) require an iban, otherwise Eupago rejects with IBAN_MISSING.
  • Documents that when iban is provided, the matching bic must also be provided (no derivation), otherwise Eupago rejects with BIC_INVALID.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jfrosorio
jfrosorio merged commit cdcf9c4 into master Jul 17, 2026
14 checks passed
@jfrosorio
jfrosorio deleted the docs-76 branch July 17, 2026 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: clarify when refunds need an IBAN and BIC

2 participants