Slowmybrosh / TFG-DietPlanner

GNU General Public License v3.0
1 stars 0 forks source link

Documentación - Introducción y Estado del arte #10

Closed Slowmybrosh closed 1 year ago

Slowmybrosh commented 1 year ago

Se han cambiado la mayoría de placeholder de la plantilla. Además se han añadido las secciones:

JJ commented 1 year ago

Todos los PRs tienen que resolver uno o varios issues, y estar en un milestone.

JJ commented 1 year ago

Trata de hacer los mensajes de commit un poco más informativos. Aunque digas "se avanza", di algo más: documentación, se añaden detalles... Además de que, en general, no "se avanza" una historia de usuarios directamente. Tienes que desarrollarla en issues.

JJ commented 1 year ago

A ver, que estuviste en IV. Problemillas

  1. Sigues sin enlazar ni issues ni de estos a HUs. (Lo has hecho en los mensajes de commit, pero también en el cuerpo del PR, puedes editarlo y añadirlos; debe quedar claro cuál es el ámbito del PR)
  2. No puedes empezar por el milestone 6. Se empieza por el 0, que es esencialmente infraestructura para que todo vaya bien o introducción.
  3. NO has contestado a alguno de los comentarios.
Slowmybrosh commented 1 year ago

A ver, que estuviste en IV. Problemillas

  1. Sigues sin enlazar ni issues ni de estos a HUs. (Lo has hecho en los mensajes de commit, pero también en el cuerpo del PR, puedes editarlo y añadirlos; debe quedar claro cuál es el ámbito del PR)
  2. No puedes empezar por el milestone 6. Se empieza por el 0, que es esencialmente infraestructura para que todo vaya bien o introducción.
  3. NO has contestado a alguno de los comentarios.

Lo de los milestone es porque la documentación es como lo último que se iba a completar paralelamente con el desarrollo del proyecto y los comentarios es debido a cerrar las conversaciones cuando no tienen discusión y cambiar lo demás porque no hay mucho que discutir.

El primer punto es totalmente cierto que debería añadir un comentario detallando todos los cambios realizados, enlazando tanto la HU como los issues trabajados. Y es una cosa que se me ha estado olvidando

JJ commented 1 year ago

A ver, que estuviste en IV. Problemillas

  1. Sigues sin enlazar ni issues ni de estos a HUs. (Lo has hecho en los mensajes de commit, pero también en el cuerpo del PR, puedes editarlo y añadirlos; debe quedar claro cuál es el ámbito del PR)
  2. No puedes empezar por el milestone 6. Se empieza por el 0, que es esencialmente infraestructura para que todo vaya bien o introducción.
  3. NO has contestado a alguno de los comentarios.

Lo de los milestone es porque la documentación es como lo último que se iba a completar paralelamente con el desarrollo del proyecto y los comentarios es debido a cerrar las conversaciones cuando no tienen discusión y cambiar lo demás porque no hay mucho que discutir.

La documentación forma parte de todos y cada uno de los PMVs, tienes que documentarlo o no es válido. Igual puedes reconsiderar que sea un milestone. Recuerda que los milestones no son tareas, son productos que entregas al cliente/clientes.

Slowmybrosh commented 1 year ago

A ver, que estuviste en IV. Problemillas

  1. Sigues sin enlazar ni issues ni de estos a HUs. (Lo has hecho en los mensajes de commit, pero también en el cuerpo del PR, puedes editarlo y añadirlos; debe quedar claro cuál es el ámbito del PR)
  2. No puedes empezar por el milestone 6. Se empieza por el 0, que es esencialmente infraestructura para que todo vaya bien o introducción.
  3. NO has contestado a alguno de los comentarios.

Lo de los milestone es porque la documentación es como lo último que se iba a completar paralelamente con el desarrollo del proyecto y los comentarios es debido a cerrar las conversaciones cuando no tienen discusión y cambiar lo demás porque no hay mucho que discutir.

La documentación forma parte de todos y cada uno de los PMVs, tienes que documentarlo o no es válido. Igual puedes reconsiderar que sea un milestone. Recuerda que los milestones no son tareas, son productos que entregas al cliente/clientes.

Si, voy a eliminar el milestone de documentación y asignar el PR a M0 ya que se trata de la introducción del proyecto

Slowmybrosh commented 1 year ago

En los últimos cambios se han introducido las siguientes mejoras:

Slowmybrosh commented 1 year ago

@JJ Esto está mejor?

JJ commented 1 year ago

@JJ Esto está mejor?

Vamos a verlo

JJ commented 1 year ago

En los últimos cambios se han introducido las siguientes mejoras:

* Se ha avanzado [[HU04] Miembro del tribunal - Criterios de Evaluación #8](https://github.com/Slowmybrosh/TFG-DietPlanner/issues/8) añadiendo una introducción y estado del arte para proponer una documentación coherente y completa del proyecto.

* Se han actualizado los objetivos ([Corregir objetivos #13](https://github.com/Slowmybrosh/TFG-DietPlanner/issues/13)) para que sean alcanzables y medibles. Teniendo en cuenta las historias de usuario propuestas y sus correspondientes milestones.

* Siguiendo [Afinar el estado del arte #11](https://github.com/Slowmybrosh/TFG-DietPlanner/issues/11), se ha eliminado todo lo sobrante en el estado del arte y se moverá a un capítulo más apropiado.

* Se ha cambiado el hiperlink a la página de FSA por una cita con la correspondiente información

Esto tiene que estar en el cuerpo del PR. La diferencia entre el cuerpo del PR y los comentarios es que el cuerpo del PR se convierte en un commit, y es entonces cuando aparecen las referencias (y cierres en su caso) de issues. Si lo pones en un comentario no sucede eso.

Slowmybrosh commented 1 year ago

@JJ Cuando puedas échale un vistazo

Slowmybrosh commented 1 year ago

@JJ He cambiado el problema siguiendo los consejos de los últimos comentarios, cambiando también los objetivos del proyecto, intentando que sean PMV. Cuando puedas échales un vistazo

Slowmybrosh commented 1 year ago

@JJ He creado las historias de usuario relativas a la resolución del problema del proyecto, cambiado el user journey intentando eliminar cualquier detalle de implementación y corregido los objetivos para reflejar el proceso de desarrollo de la solución al problema del proyecto eliminando también los posibles detalles de implementación

Slowmybrosh commented 1 year ago

@JJ He quitado todas las historias de usuarios que no corresponden a esta etapa del proyecto, creado la historia inicial de la que partirán el resto. Eliminado el user journey de la introducción para moverlo de capítulo y corregido los objetivos.

Échale un vistazo cuando puedas :)

Slowmybrosh commented 1 year ago

@JJ

Slowmybrosh commented 1 year ago

@JJ He creado los issues correspondientes a los capítulos que avanzan #8.

En el capítulo de estado del arte he cambiado alguna cosa, pero como las aplicaciones que se mencionan también se usan como recetario he optado por dejarlas. Añadiendo una línea al final sobre el objetivo del software a desarrollar.

Sobre el fichero "proyecto.tex" arreglé los cambios que hice sin darme cuenta

Slowmybrosh commented 1 year ago

Por favor, elimina del estado del arte lo que no tenga que ver. No voy a bloquear la aprobación por eso, pero hazlo

Qué consideras que no tiene que ver con el estado del arte?

JJ commented 1 year ago

Por favor, elimina del estado del arte lo que no tenga que ver. No voy a bloquear la aprobación por eso, pero hazlo

Qué consideras que no tiene que ver con el estado del arte?

El cookpad, por ejemplo, que es donde está el comentario. Es una red social basada en fotos de recetas, por lo que he entendido. Quizás alguna más, pero esa es donde he comentado.

Slowmybrosh commented 1 year ago

Por favor, elimina del estado del arte lo que no tenga que ver. No voy a bloquear la aprobación por eso, pero hazlo

Qué consideras que no tiene que ver con el estado del arte?

El cookpad, por ejemplo, que es donde está el comentario. Es una red social basada en fotos de recetas, por lo que he entendido. Quizás alguna más, pero esa es donde he comentado.

Yo si ves que no es relevante lo quito, pero es un recetario creado por la comunidad. Donde suben recetas completas (con ingredientes y pasos) con sus fotos, las comentan, etc... Pero por las veces que la he usado, funciona como un recetario normal donde encontrar cualquier tipo de receta

JJ commented 1 year ago

@Slowmybrosh OK, lo que veas. Si no te sirve el comentario, ciérralo de todas formas con "resuelto" y diciendo por qué lo haces.

Slowmybrosh commented 1 year ago

@JJ Una vez cerrados todos los comentarios, puedo hacer el merge?

JJ commented 1 year ago

@JJ Una vez cerrados todos los comentarios, puedo hacer el merge?

Está ya aprobado, así que cuando quieras