Skip to content

Conversation

@david-ruiz-cko
Copy link
Contributor

@david-ruiz-cko david-ruiz-cko commented Dec 11, 2025

This pull request introduces a new payments/setups package to support Payment Setup operations, including creating, updating, retrieving, and confirming payment setups. It also integrates the new Payment Setups client into both the ABC and NAS API entry points and adds comprehensive unit tests for the new functionality. Additionally, it improves type safety in the payments domain and updates CI workflows to use newer GitHub Actions versions.

Payment Setups feature:

  • Added a new payments/setups package with a Client that supports creating, updating, retrieving, and confirming payment setups via the API. This includes methods for each operation, with appropriate authorization handling and documentation.
  • Added a dedicated test suite in payments/setups/client_test.go to cover all Payment Setups client methods, including authorization failures and success cases.

SDK API integration:

  • Registered the new PaymentSetups client in both the abc/checkout_api.go and nas/checkout_api.go API entry points, making payment setup functionality available in both API variants. [1] [2] [3] [4] [5] [6]

Type safety improvements:

  • Updated the Ticket.IssueDate, Passenger.DateOfBirth, and Flight.DepartureDate fields in payments/payments.go from string to *time.Time to improve type safety and better represent date/time values. [1] [2] [3]

CI/CD workflow updates:

  • Upgraded GitHub Actions versions for actions/checkout and actions/setup-go in all workflow files to use the latest major versions, improving security and compatibility. [1] [2] [3]

@david-ruiz-cko david-ruiz-cko requested a review from a team December 11, 2025 15:56
@armando-rodriguez-cko armando-rodriguez-cko changed the title Feature/new endpoint payment setups feat: add new endpoint payment setups support Dec 12, 2025
@armando-rodriguez-cko armando-rodriguez-cko force-pushed the feature/new-endpoint-payment-setups branch 3 times, most recently from 6a37f54 to 0ee48fb Compare December 16, 2025 10:23
@armando-rodriguez-cko armando-rodriguez-cko force-pushed the feature/new-endpoint-payment-setups branch from 0ee48fb to d501b26 Compare December 16, 2025 10:38
@sonarqubecloud
Copy link

@armando-rodriguez-cko armando-rodriguez-cko merged commit 430f864 into master Dec 16, 2025
4 checks passed
@armando-rodriguez-cko armando-rodriguez-cko deleted the feature/new-endpoint-payment-setups branch December 16, 2025 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants