Skip to content

Comments

fix: redirect green, production, and preproduction#39

Open
JuanEstebanGarciaV wants to merge 3 commits intodevelopfrom
feature/SDK-1194-1198
Open

fix: redirect green, production, and preproduction#39
JuanEstebanGarciaV wants to merge 3 commits intodevelopfrom
feature/SDK-1194-1198

Conversation

@JuanEstebanGarciaV
Copy link
Contributor

@JuanEstebanGarciaV JuanEstebanGarciaV commented Feb 20, 2026

redirect correctly

Pull Request Documentation and Changes

Documentation of the Required Changes

  • Document the changes required for deployment.
  • Properly document each of the requirements.
  • Include project dependencies if deemed necessary.
  • Maintain this standard to comply with company requirements.

Project Details

Cells

  • Plugin and SDKs

Developers

  • Juan Esteban García Valencia

Tickets

Features

  • BUG PRODUCTION

MEJORAS DEL PR

Inventario (según estado de la transacción)
Aceptado -> Reduce inventario
Fallida -> Restaura inventario
Pendiente -> Restaura inventario
Cancelada -> Restaura inventario
Rechazada -> Restaura inventario
Abandonada -> Restaura inventario
Reintento de pago -> Restaura inventario antes de generar nueva transacción

Prevención de pagos duplicados
Validación de transaction_id único
Se ignoran callbacks repetidos (idempotencia)
No se duplica el registro del pago ni el movimiento de inventario

Sanitización de datos enviados
Se limita la longitud del nombre del producto
Se limita la longitud de la descripción
Se eliminan caracteres inválidos y HTML

Manejo de moneda
Se oculta la opción de moneda en configuración
La moneda se toma únicamente desde la factura

Formato de valores
Normalización a 2 decimales exactos
Sin separadores de miles
Compatible con validaciones de la pasarela

Errores en checkout
Se muestran errores de sesión al usuario
Evita recargas silenciosas
Facilita diagnóstico de fallos

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

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.

1 participant