[FranciscaOtero]

Description

El SDK estaba atrasado respecto de la API pública: le faltaban 33 de las 143 operaciones públicas de los specs OpenAPI de fintoc-docs (main-api.json y core-api.json). Este PR agrega las 33. Con eso el SDK cubre todo el spec público.

El inventario del SDK se armó llamando cada método de cada manager contra un HTTP mockeado y registrando el método y el path (no a partir del README). No encontré operaciones que el SDK ya tuviera con un método HTTP distinto al del spec.

Operaciones que faltaban

Implementé todas. No dejé ninguna fuera.

v1

• PATCH /v1/charges/{id}: charges.update
• POST /v1/charges/{id}/cancel: charges.cancel
• GET /v1/institutions: institutions.list
• GET /v1/invoices/{id}: invoices.get (también desde link.invoices)
• POST /v1/link_intents: link_intents.create
• GET /v1/links/exchange: links.exchange. El link que devuelve ya trae su link_token para pedir sus recursos.
• GET /v1/payouts: payouts.list
• GET /v1/payouts/{id}: payouts.get
• GET /v1/payouts/{id}/resources: payouts.resources.list. Devuelve payment intents, charges o refunds según el resource_type.
• GET /v1/refunds/{id}/voucher_url: refunds.voucher_url
• GET /v1/tax_statements: tax_statements.list (también desde link.tax_statements)
• GET /v1/tax_statements/{id}: tax_statements.get

v2

• POST /v2/entities/{entity_id}/onboardings/{id}/documents/{slot_key}: entities.onboardings.append_document
• DELETE /v2/entities/{entity_id}/onboardings/{id}/documents/{slot_key}/{document_id}: entities.onboardings.delete_document
• GET /v2/entities/{id}/settlement_account: entities.settlement_account
• GET /v2/events: events.list
• GET /v2/events/{id}: events.get
• POST /v2/invoices: invoices.create
• PATCH /v2/invoices/{id}: invoices.update
• POST /v2/invoices/{id}/finalize: invoices.finalize
• POST /v2/invoices/{id}/mark_uncollectible: invoices.mark_uncollectible
• POST /v2/invoices/{id}/pay: invoices.pay
• POST /v2/invoices/{id}/void: invoices.void
• POST /v2/payment_intents/check_eligibility: payment_intents.check_eligibility
• POST /v2/payment_intents/{id}/cancel: payment_intents.cancel
• GET /v2/payment_intents/{id}/voucher_url: payment_intents.voucher_url
• POST /v2/payment_methods: payment_methods.create, como multipart/form-data. Los campos anidados van con corchetes y los documentos de consentimiento se pueden pasar como path o como archivo.
• POST /v2/payment_methods/{id}/cancel: payment_methods.cancel
• GET /v2/payment_methods/{id}/mandate_document: payment_methods.mandate_document
• PATCH /v2/products/{id}: products.update
• GET /v2/setup_intents/{id}: setup_intents.get
• DELETE /v2/subscriptions/{id}/payment_method: subscriptions.detach_payment_method
• GET /v2/transfers/{id}/receipt: transfers.receipt

Operaciones del SDK que no están en el spec

No las toqué:

• POST /v1/checkout_sessions
• GET /v1/checkout_sessions/{id}
• POST /v1/checkout_sessions/{id}/expire
• POST /v1/payment_intents/check_eligibility (el spec la define solo en v2)

Consideraciones

• payment_methods.create codifica los campos anidados con corchetes (mx_direct_debit[consent_documents][contract]). El spec usa esa notación para los documentos, pero no lo dice explícitamente para el resto de los campos anidados.
• add_lines, remove_lines (invoices v2) y cancel (subscriptions v2) devuelven un recurso cuyo update() apunta a la URL de la acción. Esto ya pasaba en subscriptions, y ahora también en invoices porque se puede actualizar. Las acciones nuevas no tienen este problema. Lo dejé para un PR aparte.

Tests

• Tests de integración para cada operación nueva, incluyendo la clase del recurso que devuelven y el update() después de las acciones.

Requirements

None.

Additional changes

• Ahora Refund y Product (v2) se exportan como recursos. Antes los refunds y los productos se devolvían como recurso genérico.
• Ejemplos nuevos en el README: links, payouts, payment methods y documentos de onboardings.

🤖🐙 Created with Compose · Spawned by session ff794820617cf95ab458daf6f1d93c8f for Francisca Otero (francisca.otero@fintoc.com) · Requested from the web · View session

[tlarrain]

Revisaría lo del lint