Skip to content

feat: enhance PointOfSaleDetails with additional fields#469

Merged
janpaepke merged 4 commits intomasterfrom
feature/enhance-point-of-sale-details
Jan 14, 2026
Merged

feat: enhance PointOfSaleDetails with additional fields#469
janpaepke merged 4 commits intomasterfrom
feature/enhance-point-of-sale-details

Conversation

@janpaepke
Copy link
Collaborator

Summary

  • Add new fields to PointOfSaleDetails: maskedNumber, cardFingerprint, cardFunding, feeRegion, receipt
  • Add new PointOfSaleReceipt interface with receipt details
  • Update cardLabel to include Vpay and Maestro
  • Update documentation links to current API reference format

This PR replaces #466 by @cervantes-x because of #387.

References

…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.
- Added 'contactless' as a valid option for the cardReadMethod field in the PointOfSaleReceipt interface.
@janpaepke
Copy link
Collaborator Author

Note for reviewer: I already reviewed this in #466 .

Copy link
Collaborator

@edorivai edorivai left a comment

Choose a reason for hiding this comment

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

Already reviewed by @janpaepke

@janpaepke janpaepke merged commit ac7df2b into master Jan 14, 2026
7 of 11 checks passed
@janpaepke janpaepke deleted the feature/enhance-point-of-sale-details branch January 14, 2026 08:43
@janpaepke janpaepke mentioned this pull request Jan 14, 2026
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.

3 participants