search

RafaelT00 / AutoIV

Prácticas IV
GNU General Public License v3.0
0 stars 0 forks source link

[M0-Dev] Cómo van a estar relacionadas Empleado / Vacaciones con el calendario. #16

Open ignaciotitos opened 1 year ago

ignaciotitos commented 1 year ago

En el Milestone 0 y la #7 aparece la necesidad de un calendario para saber los días asignados, ¿Cómo van a estar relacionados estas estructuras? Ya que propones que calendario tiene que ser una clase aparte, que aparezca el empleado los días en el calendario que tiene aplicados es como lo entiendo. Tu me dices.

RafaelT00 commented 1 year ago

Sí, pensé el calendario como una clase para almacenar las asignaciones y además así tener un seguimiento. La clase Vacaciones o AsignarVacaciones creo que cumplen bien esto.

ignaciotitos commented 1 year ago

Vale perfecto.

danielsp13 commented 1 year ago

Todos los issues necesitan estar asociado a una de las Historias de Usuario del proyecto. Así que, en el cuerpo del issue, especifica que este problema lo has extraido del análisis de dicha HU. Si no, no es posible validar el código que hagas.

ignaciotitos commented 1 year ago

Hecho! Gracias!

JJ commented 1 year ago

Por favor, consultad las diferentes sesiones y los errores frecuentes en este objetivo. Los issues no son un intercambio entre las dos personas, sino un planteamiento de un problema, que se tiene que resolver en código. Os lo ha indicado @danielsp13 , sería conveniente que le hicierais caso. Yo en este issue no veo ningún problema planteado, y menos una solución en el código que se refiere a esto.

ignaciotitos commented 1 year ago

Ok. Cierro el issue.

JJ commented 1 year ago

Lo he dicho varias veces, pero lo repetiré. No se pueden cerrar issues desde el interfaz. Siempre desde un commit. Si no, @RafaelT00 tendrá error en sus tests siempre a partir de ahora.

ignaciotitos commented 1 year ago

Perdón. ¿Debería de volver a abrirlo o lo dejo así?

JJ commented 1 year ago

Lo he dicho varias veces, pero lo repetiré. No se pueden cerrar issues desde el interfaz. Siempre desde un commit. Si no, @RafaelT00 tendrá error en sus tests siempre a partir de ahora.

¿Se te ocurre otra forma de solucionarlo que no sea abrirlo y cerrarlo correctamente?

ignaciotitos commented 1 year ago

La verdad que no.

ignaciotitos commented 1 year ago

Supongo que poniendo la etiqueta invalid como se ha hecho anteriormente, pero no lo se.

JJ commented 1 year ago

Puedes poner la etiqueta que quieras. El tema es que tienes que volver a abrirlo y cerrarlo desde un commit, como cualquier otro issue.

ignaciotitos commented 1 year ago

Okey. Así lo haré.