Skip to content

Conversation

@david-ruiz-cko
Copy link

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

This pull request introduces support for Payment Setups to the SDK, including new client and model classes, updates to the main API, and comprehensive documentation and tests. It also improves code quality enforcement and developer setup instructions.

Payment Setups Feature:

  • Added new models and entities for Payment Setups, including PaymentSetupsRequest, customer and payment method classes, in checkout_sdk/payments/setups/setups.py.
  • Implemented the PaymentSetupsClient in checkout_sdk/payments/setups/setups_client.py, providing methods to create, update, retrieve, and confirm payment setups.
  • Registered the new client in the main API (checkout_sdk/checkout_api.py), making it accessible via api.setups. [1] [2]
  • Added integration and unit tests for the Payment Setups client in tests/payments/setups/payment_setups_client_test.py and tests/payments/setups/payment_setups_integration_test.py. [1] [2]
  • Updated the __init__.py to expose the new client.
  • Verified client instantiation in main API tests.

Developer Experience Improvements:

  • Enhanced the README.md with clearer requirements, installation, development setup, testing, code quality instructions, and environment variable documentation. [1] [2]
  • Added flake8 pre-commit hook configuration to .pre-commit-config.yaml for improved code quality checks.

@david-ruiz-cko david-ruiz-cko requested a review from a team December 11, 2025 16:06
david-ruiz-cko and others added 4 commits December 18, 2025 23:46
…ups' into feature/new-endpoint-payment-setups

# Conflicts:
#	checkout_sdk/payments/setups/setups.py
#	checkout_sdk/payments/setups/setups_client.py
#	tests/payments/setups/payment_setups_client_test.py
#	tests/payments/setups/payment_setups_integration_test.py
@armando-rodriguez-cko armando-rodriguez-cko changed the title Feature/new endpoint payment setups feat: add payment setups support Jan 20, 2026
@sonarqubecloud
Copy link

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.

3 participants