Closed pablo1mc315 closed 2 years ago
Buenas @alvarogaro. Este es el PR donde irán apareciendo las modificaciones que realice sobre el proyecto para crear la estructura de datos. Te iré abriendo varios ISSUES para ir discutiendo cuestiones acerca del proyecto :construction: 💪🏼
Buenas @alvarogaro, échale un ojo a ver que te parece lo que llevo de momento y cualquier cosa me dices (si encuentras algún getter que no sea necesario o si algún atributo no se ajusta a lo que deseas).
También necesito que añadas los issues que no lo estén, así como este PR, al M0 para cuando realice la entrega y también que encuentres una base de datos cuanto antes, como te digo en #9, para hacer que la estructura de datos se ajuste lo máximo posible a ella. Un saludo! 😶🌫️ 🐧
Respecto a la base de datos creo que para este objetivo no va a ser necesario ( De todas formas lo hablaré con JJ) , con que se ajuste a los requisitos de las historias de usuario será suficiente. He estado echándole un ojo al PR y la verdad que en un principio lo veo bien, excepto el enumerado que te lo he comentado el resto lo veo bien, de todas formas ya tendría el profesor que echarle un vistazo a ver que tal lo ve
@JJ listo para revision
Lista de Comprobación
Prerrequisitos
- [x] Se tendrá que haber avanzado en el objetivo 1 antes de poder entregar este.
Sobre la estructura del repositorio
- [x] ¿He seguido las mejores prácticas en nombre de las clases y ficheros y disposición de los mismos?
Sobre el análisis del problema
- [x] ¿Se ha documentado qué análisis se ha hecho sobre el dominio para decir lo que se ha creado? Sí, en [M0 - Dev] Objetivo de la aplicación #8.
- [ ] ¿Se ha documentado por qué se ha elegido que lo creado sea un objeto valor, una entidad o un agregado?
En la misma configuración del repo se tiene que decir qué entidad se ha programado, en el iv.yaml. Si no lo es, ¿qué fichero has puesto?
Si no has seguido DDD, ¿qué metodología se ha seguido?
En el iv.yaml aparece qué entidades se han programado y cuales son los objetos valor. No sabía que esa casilla de la lista de comprobación se refería a eso, así que no la marqué, fallo mío, procedo a marcarla @JJ
Se han enlazado los issues con la HU y se han reabierto los que estaban cerrados (fallo mío), los cuales se cerrarán cuando se haga mergee el PR en el commit correspondiente o en el cuerpo del mismo. Listo para revisión @JJ
@JJ listo para revision de nuevo! 🙂
@alvarogaro listo para mergear cuando quieras
Lista de Comprobación
Prerrequisitos
Sobre la estructura del repositorio
Sobre el análisis del problema
Sobre la planificación y la programación
closes #8 closes #9 closes #10 closes #11