ciaolavoro / ciao-lavoro

💼 Aplicación web para servicios independientes
https://ciaolavoro-s3.pages.dev
6 stars 0 forks source link

Fix 31 - Contrato #441

Closed alisanhos closed 6 months ago

alisanhos commented 6 months ago
Samalbalat commented 6 months ago

Sobre este punto: No deja poner decimales en la creación de contrato. Se ha testeado con 10 , 10,0 y 10.0 y todos los valores funcionan

Samalbalat commented 6 months ago

Sobre este punto: Revisar si es posible crear un contrato con fecha anterior a la actual. Revisen quien ha realizado las restricciones de frontend porque para ellos coge una hora que no es la hora del PC de uso hasta donde me he percatado, mucho ojo con eso por lo demás, no, no se puede hacer

Samalbalat commented 6 months ago

Sobre este punto: Revisar si la búsqueda de contratos funciona correctamente. Los rangos de fechas no funcionaban y tenían ángulo de error porque en front si no se especificaba siempre se pasaba la fecha como '', ahora se comprueba dicho asunto y encima se ha realizado mejorado el listado en este commit 0f636be5c000b282e87db08ca4ed1ae7c4c4f3ab

Samalbalat commented 6 months ago

Sobre este punto: Es posible crear un contrato que inicie dentro de 1000 años y dura 10 años. Realizar validaciones más estrictas. Se ha establecido un máximo de 8 horas por contrato y una dilatación máxima respecto al inicio de 6 meses https://github.com/ciaolavoro/ciao-lavoro/commit/ae0061b0838206c89ddbc79a80e0462a354ccc7e

Samalbalat commented 6 months ago

Sobre este punto: Al pagar un contrato, da un error al poner 0 puntos o puntos decimales. También deja poner -0 puntos y valores no numéricos. El campo en front debería de ser un campo númerico como en la creación de contrato, este tampoco acepta valores negativos pero si debería de aceptar 0 y enviar un JSON como en la creación de contrato, en esta segunda función incluso acepta las comas para poder pasarlo en backend a float y de float a integer o alternativamente impedir los valores decimales en front

Sin importar eso se ha alterado en backend la línea que lo gestiona para que incluso con decimales se reciba un valor entero Commit aquí https://github.com/ciaolavoro/ciao-lavoro/commit/f0a78155bc3eae4d02890268a7697c1f045e749d

Samalbalat commented 6 months ago

Sobre este punto: Revisar si existe algún error al indicar un contrato como terminado. No aparentemente pido segunda opinión a @alvhidrod

Samalbalat commented 6 months ago

Sobre este punto: Revisar si es posible cancelar un contrato. No aparentemente pido segunda opinión a @alvhidrod

Samalbalat commented 6 months ago

Sobre este punto: Si tengo más de un contrato con un trabajador, al pagar uno de los contratos se paga automáticamente el otro. Revisar si está relacionado con los estados de los contratos.

No sé como están echas las llamadas al backend en front end pero aprecer ser que se ejecutan 2 PUTs por cada contrato al procesar el pago, habría que revisar dicha parte

alisanhos commented 6 months ago

Desde frontend en Crear Contrato ya están solucionadas varias cosas:

Image

Image

alisanhos commented 6 months ago

He conseguido arreglar todo lo respecto a Contrato, lo único que no consigo solucionar es el tema del pago. Enviaré un video por discord para que podais verlo

alisanhos commented 6 months ago

Disculpad por el último commit, eran cambios correspondientes a otra rama. Por algún motivo no hice bien el cambio de rama por comandos.

alvhidrod commented 6 months ago

Revisadas las dos funcionalidades pedidas por @Samalbalat, funcionan como se debe