Correcciones - Grupo 5

[Bruno Pistarino]

Buenas noches profes, ya realizamos las correcciones pero siguen estando como pendientes.

• [Entrega 3] Está roto postear feedback sin estar logueado (se limpia el form, pero no se guarda).
• [Entrega 4] Falta la feature de poder configurar si se admiten votos anónimos o no.

[fisa]

Buenas! Como comentamos el martes, solo quedaba "programada" la corrección de la entrega 5. A todo lo demás ya estamos en modo "corregimos cuando nos avisan que está arreglado". Así que ahora que avisaron lo miro!

Saludos!

--
You received this message because you are subscribed to the Google Groups "UCSE SEIA" group.
To unsubscribe from this group and stop receiving emails from it, send an email to ucse-seia+...@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/ucse-seia/6923d8c3-09d8-4153-a308-26471f1556bcn%40googlegroups.com.

--

fisa  -  Juan Pedro Fisanotti

[fisa]

Revisado y actualizadas las correcciones!

[Agustín Duelli]

Buenas! Sobre las correcciones:

• Por las dudas, ya no deja postear feedback anónimo, solo comentarios a feedback existente. Es intencional?
  Si, el cambio es intencional. Tampoco se pueden hacer comentarios anónimos ahora!
  

Sobre la entrega 6, no vamos a poder implementar un dockerfile que pueda generar los contenedores de la aplicación y Convex al mismo tiempo. Esto por cuestiones de self-hosting de Convex que no se pueden resolver por nuestra parte.
La solución que encontramos fue generar dos compose: uno para Convex, otro para la aplicación de frontend. Agregamos un paso a paso en el README del repo con los pasos adicionales.

Todavía no está la entrega subida, queríamos primero avisar del cambio.

[fisa]

Ok! Gracias por la respuesta al comentario.

Sobre la entrega 6: tiene que funcionar con los comandos que pusimos en la consigna, porque la corrección de eso es bastante automática, ya que si no sería un poco inescalable tener que aprendernos cómo buildear y correr contenedores para cada grupo.

Así que levanten lo que puedan dentro de un solo contenedor, que funcione con los comandos de la consigna y sea suficiente como para ver la web localmente y tener algún dato persistente en el volumen. Linkeen al Convex de prod para no tener que levantarlo localmente en otro contenedor, y persistan entonces alguna otra cosa que no sea la db dentro del volumen persistente (y nos tienen que indicar qué es, así sabemos cómo probar que se genere esa data).

Saludos!

To view this discussion visit https://groups.google.com/d/msgid/ucse-seia/aa885bfa-2736-4744-b9c6-ed75e1412428n%40googlegroups.com.

[Agustín Duelli]

Buen día! Sobre la entrega 6, ya está subida para corregir!

Decidimos implementar un pequeño logger de middleware. La idea es tener registro de todas las páginas visitadas por los usuarios, sin loggear datos de usuario que sean personales. Esto para poder a futuro generar métricas agrupadas, armar tableros de control, etc.
Persistimos usando una DB sqlite, con un cliente de Prisma. Se generan nuevos registros con cada página visitada por el usuario.

[Agustín Duelli]

Pequeña aclaración, por si acaso. 

- Los registros son solo de cargas de página dentro de la app. Van a tener que recorrer algunas pantallas para tener datos.
- Se pueden ver los datos también por rutas de la app! En localhost:8000/api agregamos varias rutas para verificar rapidamente:
1. localhost:8000/api/analytics/page-views. Ver registros de logger.
2. localhost:8000/api/analytics/log. Postear un log.
3. localhost:8000/api/analytics/database-info. Datos básicos de la DB
4. localhost:8000/api/analytics/project-stats. Estadísticas por proyecto (puede que falle, no está 100% implementado)

[fisa]

Buenas! Gracias por avisar y muchas gracias por dejarnos urls listas para probar la persistencia. Actualizadas las correcciones!

To view this discussion visit https://groups.google.com/d/msgid/ucse-seia/69e9f510-1ca4-4293-9a0f-8b5f55d56024n%40googlegroups.com.

[Agustín Duelli]

Buenas noches. Sobre la corrección de entrega 6.

El error para cargar el schema de la DB no lo estamos pudiendo reproducir. Probamos en 5 máquinas diferentes, windows 11 y macOS, y ninguna tuvo problemas. 

Vamos a seguir intentando resolverlo igualmente.

To view this discussion visit https://groups.google.com/d/msgid/ucse-seia/CAN7o839MFGJEBSduJYbXnaWHqLk4O0cwzbBEcGWUpKOLUktp2Q%40mail.gmail.com.

[fisa]

Buenas!

Por las dudas, los comandos que estoy corriendo son los que figuran en la consigna, nada extra. Si hay que correr cosas extras por ejemplo para crear la db la primera vez, deberían idealmente hacerlo de forma automática en el container (por ejemplo varios grupos de django corren el migrate en el start del container, lo que crea automaticamente la db si no existe). O en su defecto avisarme de forma sencilla qué debo hacer para que se cree la db la primera vez (no ideal pero se los aceptamos). Pegué una mirada al README, pero tiene unas 300 líneas de cosas :D

Saludos!

To view this discussion visit https://groups.google.com/d/msgid/ucse-seia/CAN5eBTN8Rwzz3E4BsY9aejs-orxO3A2%3DSA%3DtqLOsaRV3HaEiUg%40mail.gmail.com.

[fisa]

For the record: era un tema de permisos que funcionan diferente entre Mac y Linux, por estar usando un usuario custom dentro del container.

Lo solucioné dando super permisos al volumen (chmod 777).

Actualizadas las correcciones!