Skip to content

feat: enhance PointOfSaleDetails with additional fields and update documentation links#466

Closed
cervantes-x wants to merge 3 commits intomollie:masterfrom
cervantes-x:feature/enhance-point-of-sale-details
Closed

feat: enhance PointOfSaleDetails with additional fields and update documentation links#466
cervantes-x wants to merge 3 commits intomollie:masterfrom
cervantes-x:feature/enhance-point-of-sale-details

Conversation

@cervantes-x
Copy link
Contributor

  • Added new fields: maskedNumber, cardFingerprint, feeRegion, and receipt to the PointOfSaleDetails interface.
  • Updated existing documentation links to reflect the latest API references.
  • Updated cardLabel to include 'Vpay' and 'Maestro' options.

…cumentation links

- Added new fields: maskedNumber, cardFingerprint, feeRegion, and receipt to the PointOfSaleDetails interface.
- Updated existing documentation links to reflect the latest API references.
- Updated cardLabel to include 'Vpay' and 'Maestro' options.
- Introduced a new optional field `cardFunding` to the PointOfSaleDetails interface, indicating the card funding type (credit or debit) if known.
- Updated documentation to reflect the new field and its possible values.
Copy link
Collaborator

@janpaepke janpaepke left a comment

Choose a reason for hiding this comment

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

Thanks for this PR! The new fields align well with the current Mollie API docs.

One issue: cardReadMethod is missing the contactless value.

The docs list:

chip | magnetic-stripe | near-field-communication | contactless | moto

Should be:

cardReadMethod?: 'chip' | 'magnetic-stripe' | 'near-field-communication' | 'contactless' | 'moto';

@janpaepke janpaepke added the API discrepancy Inconsistency between Mollie's REST API and this SDK. label Dec 22, 2025
@janpaepke janpaepke requested a review from Copilot December 22, 2025 21:17
Copy link
Contributor

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 enhances the PointOfSaleDetails interface with additional payment information fields and updates all documentation links to reference the latest Mollie API documentation.

Key changes include:

  • Added four new fields to capture more detailed payment information: maskedNumber, cardFingerprint (relocated), feeRegion, and receipt
  • Introduced a new PointOfSaleReceipt interface to provide detailed transaction receipt information
  • Updated cardLabel type to include 'Vpay' and 'Maestro' options

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

- Added 'contactless' as a valid option for the cardReadMethod field in the PointOfSaleReceipt interface.
@janpaepke
Copy link
Collaborator

Thank you for the contribution! Replaced by #469 because of #387 (CI can't run on forked PRs).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API discrepancy Inconsistency between Mollie's REST API and this SDK.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants