Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README y funcionalidades fronted #75

Open
MariaJoseSilva opened this issue Oct 2, 2024 · 11 comments
Open

README y funcionalidades fronted #75

MariaJoseSilva opened this issue Oct 2, 2024 · 11 comments
Assignees
Labels

Comments

@MariaJoseSilva
Copy link

MariaJoseSilva commented Oct 2, 2024

Hola!
Respecto al README

  • Esta vez no es necesario incluir una explicación del código como las tareas anteriores?
  • En la sección Sequelize solo se incluyen los comandos que se corren en consola? O también hay que describir los cambios "manuales" que se le aplican a las migraciones y modelos?

Respecto al fronted

  • Hay algunos endpoints que no encuentro en el fronted, como por ejemplo, editar o eliminar un mensaje. Estos casos se van a revisar con Postman?

Gracias!

@c4ebt
Copy link

c4ebt commented Oct 2, 2024

Hola! Tengo una duda similar, por lo que me pareció mejor hacerla aquí en vez de hacer una nueva issue.
En los comandos de sequelize, ¿es necesario poner todos los comandos de generación de modelos y migraciones? Entiendo que haya que poner los de correr las migraciones y las seeds, pero me hace ruido que haya que poner los de generación porque son comandos que al final solo crean código que igual está disponible en el repositorio de entrega.

@rodrigoogalde
Copy link
Member

Hola!
@MariaJoseSilva

  1. No, no es necesario
  2. Solo los de la consola.
  3. En efecto, no están implementados en el front, por lo que lo puedes probar con postman.

@rodrigoogalde
Copy link
Member

rodrigoogalde commented Oct 2, 2024

@c4ebt
Sip, son:

  • 3 comandos, para generar cada modelo.
  • 1 para migrar
  • 3 para las seeds

@rodrigoogalde rodrigoogalde self-assigned this Oct 2, 2024
@c4ebt
Copy link

c4ebt commented Oct 2, 2024

@rodrigoogalde Gracias! Solo para tenerlo más claro, ¿te refieres a 3 comandos para generar cada modelo, o 1 comando por cada modelo, en total 3 comandos de generación de modelos?

@rodrigoogalde
Copy link
Member

Solo puedes usar 1 comando por modelo, por lo tanto son 3 comandos

@panchouc
Copy link

panchouc commented Oct 3, 2024

Hola @rodrigoogalde, si yo generé mis modelos de forma manual, es decir, escribí el archivo del modelo en si y de ahí solo hice las migraciones y seed, está correcto igualmente? A lo que me refiero, es que en mi caso el ayudante no tiene que ejecutar ningún comando para crear modelos, ya que yo cree los modelos a mano. Lo anterior, ya que necesitaba definir ciertas llaves foráneas y primarias de manera customizada.

@panchouc
Copy link

panchouc commented Oct 3, 2024

Lo otro, me di cuenta recién que dices que son 3 comandos para las seeds, ¿Qué pasa si lo hice como en la ayudantía que ocupo yarn sequelize-cli db:seed:all, está correcto igual?. ¿Se permite flexibilidad mientras esté todo especificado en el readme?

@rodrigoogalde
Copy link
Member

Hola @panchouc
No estoy seguro a que refieres a mano. Para hacer los modelos, se utiliza el comando de sequelize-cli.
Con respecto a tu otra duda, es válido usar ese comando

@panchouc
Copy link

panchouc commented Oct 3, 2024

Hola @rodrigoogalde, es que por ejemplo en la documentación de sequelize aparece esto para generar un modelo ***, pero lo que yo hice, siguiendo con este ejemplo, es que en la carpeta models directamente llegué y cree un archivo user.js y ahí escribí yo mismo el código para el modelo, y de ahí lo que hice fue crear una migración para el modelo con el comando ** NOMBRE_MIGRACION, entonces no ocupé en ningún momento el comando de sequelize para crear el modelo, solo para la migración. Me dices si es que se entiende mejor así.

@rodrigoogalde
Copy link
Member

Como te comenté anteriormente, el comando de sequelize te crea los modelos y las migraciones. Si los hiciste a mano y funciona bien, no debería haber problema. Ten en cuenta que el comando de sequelize también crea una migración.
Lo que deberías hacer es usar el comando y luego modificar el modelo según lo que se indica el enunciado (agregar FK, PK, etc)

@panchouc
Copy link

panchouc commented Oct 3, 2024

Ah okay, muchas gracias Rodrigo, ahí me queda más claro. Si mi problema era ese, que no sé como agregar las FK, PK a los modelos, entonces como que había decidido programarlo yo de cero el modelo y posteriormente crear la migración para ese modelo en específico. Muchas gracias

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants