[IV-22-23] Objetivo 2 #10

vdeq79 · 2022-10-14T17:44:41Z

Sobre la estructura del repositorio

¿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

¿Se ha documentado qué análisis se ha hecho sobre el dominio para decir lo que se ha creado?
¿Se ha documentado por qué se ha elegido que lo creado sea un objeto valor, una entidad o un agregado?
¿He propuesto un producto mínimamente viable, que en muchos casos será un solo objeto valor que no dependa de ningún otro (y que sea la base de muchos otros)?.

Sobre la planificación y la programación

¿Los issues representan un problema, y no una tarea?
¿Todos los mensajes de commit explican el cambio, y no se limitan a repetir el nombre del fichero que se ha cambiado?
¿Los mensajes de commit siguen el formato estándar y buenas prácticas?
¿Se ha hecho una revisión real del código para comprobar que todos los atributos y funciones creadas están respaldadas por una HU?
¿Todos los issues creados están asignados a una HU?
¿Todos los cambios en el código están asignados a un issue al que se referencia en un commit? Los issues sobre los que estoy trabajando, ¿han sido asignados al primer milestone?
¿Se ha asignado al mismo milestone el PR que se ha hecho?
¿Son los milestones sobre los que estoy trabajando productos mínimamente viables? ¿O tengo que solicitar al product manager que cree milestones adicionales o precise de qué producto se trata?

…racion.ts

…tacion de clases

arsa-dev · 2022-10-20T07:29:27Z

models/orden_de_fabricacion.ts

+    modelos: Array<[Modelo, number]>;
+
+    constructor(modelos:Array<[Modelo, number]>){


Suggested change

modelos: Array<[Modelo, number]>;

constructor(modelos:Array<[Modelo, number]>){

modelos: Array<ModeloFabricacion>;

constructor(modelos:Array<ModeloFabricacion>){

@vdeq79 aquí creo que sería más conveniente hacer un objeto valor intermedio, ya que de ese modo se podrá identificar que number (unidades) es de qué Modelo y además así será más sencillo de modelar en un futuro en la base de datos en la propia tabla de relación N:N.

Bastaría con crear ese objeto valor con dos propiedades, la primera el modelo y la segunda numérica para las unidades.

Quieres decir que un modelo puede estar en distintos órdenes de fabricación? Es que no acabo de entender muy bien el concepto de orden de fabricación.

Sí, un mismo modelo podría fabricarse varias veces, por ejemplo en el caso de una fábrica de camisas:

Una operación puede ser: Coser botones (una misma operación también puede estar en varios modelos)

Un modelo puede ser: Camisa de rayas (que tendrá una serie de operaciones como la anterior)

Una orden de fabricación es simplemente la orden que se le pasa a los operarios de que hay que fabricar, por ejemplo:

5 camisas de rayas (con x operaciones)

20 camisas de cuadros (con y operaciones)
Y en una segunda orden de fabricación se podrían volver a fabricar camisas de rayas, por tanto un mismo modelo puede estar en varias órdenes de fabricación

vdeq79 · 2022-10-20T07:46:50Z

Por cierto, agradecería si pudieras añadir este PR a tu M0 para poder pasar los tests del repositorio de @JJ.

arsa-dev · 2022-10-20T07:31:24Z

models/modelo.ts

+import { Tiempo } from "./tiempo.ts";
+import { Operacion } from "./operacion.ts";


Suggested change

import { Tiempo } from "./tiempo.ts";

import { Operacion } from "./operacion.ts";

import { Tiempo } from "./tiempo";

import { Operacion } from "./operacion";

Los imports en typescript no deben de incluir la extensión de archivo

La razón por la que he puesto las extensiones es porque he usado el runtime recomendado Deno y para Deno, es necesario explicitar la extensión .ts (denoland/deno#2506). Si tienes pensado utilizar otro runtime lo cambio sin problemas.

@vdeq79 he abierto una issue donde discuto la elección del runtime, a modo de resumen creo que es más conveniente el uso de NodeJS frente a Deno, no obstante, puedes añadir cualquier cosa que consideres oportuna. Sino, esta conversación closes #12

Yo os animaría a que usárais deno. Pero es cuestión vuestra.

¿@JJ has revisado la issue #12? Me gustan los conceptos sobre los que nace Deno, aunque no lo veo tan madurado como para utilizarlo en el proyecto. Encuentro poco soporte y bastantes problemas de compatibilidad de elementos muy extendidos, por ejemplo si se quiere incluir en un futuro algún orm (uno de los más extendidos es typeorm) pues nos encontramos con este otro problema de compatibilidad: typeorm/typeorm#6123 - https://stackoverflow.com/questions/68413239/how-to-import-typeorm-in-deno

Quizás si que lo usaría para alguna solución mucho más concreta donde no se pretenda extender el producto para usar estas tecnologías que comento en la issue.

No vais a usar ORM en este proyecto. Y con deno tienes cosas como deno deploy que pueden ser muy interesantes como PaaS.

Sí, lo comentaba como otro ejemplo más de algo que suele haber en cualquier servicio, aunque creo que ya lo comentamos hace tiempo en clase que no íbamos a entrar en ese tema en la asignatura. Pero los otros que comento en la issue como la generación de openapi a través de metadatos en modelos y controladores, etc. Si los considero buenas prácticas para las que actualmente en deno sería posible aplicarlas pero a más bajo nivel. Además, aunque no tenga calificación en la asignatura, posteriormente me gustaría continuar el proyecto usando la misma base de código (aunque por supuesto si no es posible tengo el foco principalmente en superar objetivos).

Por cierto he estado revisando también deno deploy, y sí efectivamente se ve muy interesante aunque aún viendo como otro punto a su favor no lo veo algo decisivo porque pienso que aunque sea muy buena opción, cuando se pone una solución en manos de un tercero para alojarlo siempre debemos de tener una segunda opción lista en un plan de recuperación de desastres, por tanto también habría que evaluarlas y desarrollarlas, ya que aunque nos facilite deno el desplegar soluciones por el hecho de usar deno en sí, también debemos como mínimo conocer como hacerlo sin deno deploy.

models/modelo.ts

arsa-dev · 2022-10-22T10:15:36Z

Por cierto, agradecería si pudieras añadir este PR a tu M0 para poder pasar los tests del repositorio de @JJ.

Añadido @vdeq79

…en los imports

arsa-dev

Listo para la revisión por parte de @JJ

JJ

👍

vdeq79 mentioned this pull request Oct 16, 2022

Implementación de clases para el modelo #9

Closed

4 tasks

vdeq79 force-pushed the xu-objetivo-2 branch from ba1427f to 2787a04 Compare October 16, 2022 09:21

vdeq79 added 2 commits October 16, 2022 11:22

Avanza la tarea arsa-dev#9 con la creacion de la carpeta models y ope…

fe18be2

…racion.ts

Avanza la tarea arsa-dev#9 con la primera aproximacion en la implemen…

321ef10

…tacion de clases

vdeq79 force-pushed the xu-objetivo-2 branch from 2787a04 to 321ef10 Compare October 16, 2022 09:23

vdeq79 added 3 commits October 16, 2022 11:36

Avanza la tarea arsa-dev#9 con la modificación de tiempo.ts

eb91ea8

Avanza la tarea arsa-dev#9 añadiendo iv.yaml

feb49da

Avanza la tarea arsa-dev#9 corrigiendo iv.yaml

0866ad9

vdeq79 force-pushed the xu-objetivo-2 branch from c45f4ab to 0866ad9 Compare October 18, 2022 10:11

arsa-dev reviewed Oct 20, 2022

View reviewed changes

arsa-dev requested changes Oct 20, 2022

View reviewed changes

vdeq79 added 5 commits October 20, 2022 11:53

Avanza la tarea arsa-dev#9 cambiando la convencion a lowerCamelCase

8b2e038

Avanza la tarea arsa-dev#9 reemplazando la clase Tiempo

a8e0346

Avanza la tarea arsa-dev#9 añadiendo la clase ModeloFabricacion

d4a3b4e

Avanza la tarea arsa-dev#9 cambiando la clase OrdenDeFabricacion

dece4e5

Avanza la tarea arsa-dev#9 cambiando iv.yaml

6c8af95

arsa-dev added this to the [M0] Modelo de información mínimo milestone Oct 22, 2022

github-actions bot mentioned this pull request Oct 22, 2022

Update objetivo-2.md JJ/IV-22-23#120

Merged

3 tasks

arsa-dev approved these changes Oct 22, 2022

View reviewed changes

arsa-dev mentioned this pull request Oct 22, 2022

Elección del runtime a utilizar #12

Closed

Avanza la tarea arsa-dev#9 con la confirmación del runtime y ajustes …

cda5b91

…en los imports

arsa-dev approved these changes Oct 23, 2022

View reviewed changes

arsa-dev requested a review from JJ October 23, 2022 10:57

JJ approved these changes Oct 23, 2022

View reviewed changes

arsa-dev merged commit 7c7da59 into arsa-dev:main Oct 23, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[IV-22-23] Objetivo 2 #10

[IV-22-23] Objetivo 2 #10

vdeq79 commented Oct 14, 2022 •

edited

Loading

arsa-dev Oct 20, 2022

vdeq79 Oct 20, 2022

arsa-dev Oct 22, 2022 •

edited

Loading

vdeq79 commented Oct 20, 2022

arsa-dev Oct 20, 2022

vdeq79 Oct 20, 2022

arsa-dev Oct 22, 2022

JJ Oct 22, 2022

arsa-dev Oct 22, 2022

JJ Oct 22, 2022

arsa-dev Oct 22, 2022

arsa-dev commented Oct 22, 2022

arsa-dev left a comment

JJ left a comment

		modelos: Array<[Modelo, number]>;

		constructor(modelos:Array<[Modelo, number]>){

		import { Tiempo } from "./tiempo.ts";
		import { Operacion } from "./operacion.ts";

[IV-22-23] Objetivo 2 #10

[IV-22-23] Objetivo 2 #10

Conversation

vdeq79 commented Oct 14, 2022 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

arsa-dev Oct 22, 2022 • edited Loading

Choose a reason for hiding this comment

vdeq79 commented Oct 20, 2022

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

arsa-dev commented Oct 22, 2022

arsa-dev left a comment

Choose a reason for hiding this comment

JJ left a comment

Choose a reason for hiding this comment

vdeq79 commented Oct 14, 2022 •

edited

Loading

arsa-dev Oct 22, 2022 •

edited

Loading