Skip to content

Add verificar_intentos_pago for Bancario_Externo/Stripe routing detection#155

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/verify-connection-status
Draft

Add verificar_intentos_pago for Bancario_Externo/Stripe routing detection#155
Copilot wants to merge 2 commits intomainfrom
copilot/verify-connection-status

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 9, 2026

Payments with status == "Bancario_Externo" are direct corporate IBAN transfers that bypass Stripe entirely. Without explicit routing detection, these payments were indistinguishable from blocked/unverified ones.

Changes

verificar_conexion_real.py

  • Added verificar_intentos_pago(intentos_pago) that prints a --- [VERIFICACIÓN DE CONEXIÓN] --- header then classifies each payment:
    • Bancario_Externo → advisory that payment skips Stripe + IBAN transfer notice
    • Any other status → blocked-payment error with amount
verificar_intentos_pago([
    {"status": "Bancario_Externo", "emisor": "Corp A", "monto": 3000},
    {"status": "Pendiente",        "emisor": "Corp B", "monto": 750},
])
# --- [VERIFICACIÓN DE CONEXIÓN] ---
# AVISO: El pago de Corp A NO pasará por la App de Stripe.
# MOTIVO: Transferencia corporativa directa al IBAN.
# ERROR: Pago de 750€ bloqueado por falta de verificación.

tests/test_verificar_conexion_real.py (new)

  • 9 unit tests covering: header always present, Bancario_Externo advisory/motivo, absence of cross-path messages, error with correct monto, mixed-list ordering, and empty-list edge case.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tryonyou-app Ready Ready Preview, Comment Apr 9, 2026 3:42am
tryonyou-pilot Error Error Apr 9, 2026 3:42am

Request Review

Copilot AI changed the title [WIP] Check payment connection and status handling Add verificar_intentos_pago for Bancario_Externo/Stripe routing detection Apr 9, 2026
Copilot AI requested a review from LVT-ENG April 9, 2026 03:43
@cursor cursor bot deleted the copilot/verify-connection-status branch April 9, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants