Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/checkout expiration #97

Merged
merged 11 commits into from
Sep 11, 2024
Merged

Feature/checkout expiration #97

merged 11 commits into from
Sep 11, 2024

Conversation

andrextor
Copy link
Collaborator

No description provided.


### Nuevo:

- La propiedad `payment.expiration` en el endpoint de creación de sesión ` /api/session` cambia de ser requerida a **opcional**. Si la fecha de expiración no se envía, se asignará automáticamente un valor predeterminado de `30 minutos` a partir de la fecha de creación de la sesión. [Fecha de expiración](/checkout/expiration)
Copy link
Contributor

Choose a reason for hiding this comment

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

La propiedad no es payment.expiration si te fijas la propiedad es expiration en la raiz.

Lo que me hace pensar que los changelog deberían llevar el nombre de la entidad o estructura que se está cambiando. Para este caso es el CreateSessionRequest o al menos así lo hemos llamado en la documentación algunas veces.

Mi propuesta:

La propiedad CreateSessionRequest.expiration en el endpoint de creación de sesión /api/session cambia de ser requerida a opcional ...


# Fecha de expiración

Cuando se envía la propiedad `expiration` en la creación de una sesión esta define la fecha límite en la que una sesión debe completarse. El usuario debe finalizar el proceso antes de que se alcance esta fecha.
Copy link
Contributor

Choose a reason for hiding this comment

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

Lo mismo aplicaría aquí no?

Cuando se envía la propiedad CreateSessionRequest.expiration ...


# Fecha de expiración

Cuando se envía la propiedad `expiration` en la creación de una sesión esta define la fecha límite en la que una sesión debe completarse. El usuario debe finalizar el proceso antes de que se alcance esta fecha.
Copy link
Contributor

Choose a reason for hiding this comment

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

Necesitamos agregar una aclaración de que no necesariamente se debe finalizar, pues los pagos pendientes o cash siguen activos no?

@eduarguz eduarguz merged commit 00bde89 into main Sep 11, 2024
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