Correcciones2020 - ucseiw-team/catedra GitHub Wiki
Grupo 1 (Juego PixelArt)
| Entrega | Estado en la primer fecha | Estado en la segunda fecha |
|---|---|---|
| 1 | No aprobada, correcciones pendientes | Aprobada |
| 2 | No aprobada, detalles a corregir | Aprobada |
| 3 | No aprobada, correcciones pendientes | Aprobada |
| 4 | No aprobada, detalles a corregir | Aprobada |
| 5 | No aprobada, faltan items | Aprobada |
| 6 | Aprobada |
Correcciones pendientes:
Ninguna.
Otras observaciones:
- En sus entregas van a tener considerablemente más trabajo de frontend que los demás grupos, por lo que vamos a ir regulando para que el trabajo de backend sea un poco mejor, compensando. Pero no se cuelguen con dudas y demás, pregunten a tiempo, porque muy probablemente vayan a tener problemas que incluso para nosotros (profes) pueden llegar a ser complicados de resolver.
- Felicitaciones por sacar andando la grilla sincronizada entre usuarios, es una feature bastante fuera de lo normal para lo que enseñamos en la materia, y lo lograron con bastante poca ayuda de nuestra parte :)
- Genial la idea de denunciar regiones de la grilla!
- Consejo: no se pedía en la consigna por lo que no hace falta que lo hagan, pero sería útil que la data de MEDIA también esté en /data al utilizar docker. Fue algo extra que muchos otros grupos hicieron, y por eso se los comento como cosa interesante.
- Felicitaciones por cómo quedó funcionando el mecanismo de denuncias :)
Grupo 2 (Running)
| Entrega | Estado en la primer fecha | Estado en la segunda fecha |
|---|---|---|
| 1 | No aprobada, correcciones pendientes | Aprobada |
| 2 | No aprobada, detalles a corregir | Aprobada |
| 3 | No aprobada, correcciones pendientes | Aprobada |
| 4 | No aprobada, detalles a corregir | Aprobada |
| 5 | Aprobada | |
| 6 | Aprobada |
Correcciones pendientes:
- [E6] El settings hace que cuando se corran las cosas en el container de docker, la url de todo lo que es media esté rota (porque setea incorrectamente el MEDIA_URL, que es una url web, no un path a archivos). Como es un detalle la entrega queda aprobada, pero estaría bueno que lo dejen arreglado.
Otras observaciones:
- Hay que tener en cuenta el riesgo de gente no profesional posteando métodos de entrenamiento que puedan afectar a la salud de otra gente. Limitamos quién postea? Se cubren legalmente de alguna forma? etc.
- La UI en varias partes (especialmente eventos y entrenamientos) es un poco "sistema de gestión". Habría que ver cómo pensarlo, no es fácil, pero como está hoy en día me parece que no sería muy práctico para los usuarios. Y no me refiero a la cuestión de estética, de si es lindo o feo, sino a cosas de usabilidad y cómo mostrar la info (ej: una tabla para los entrenamientos). La parte de eventos por ejemplo tiene más sentido :)
- Por qué agregaron a mano un campo id_evento en el modelo de eventos? Porque es prácticamente idéntico a lo que les habría agregado django por default (como en todos los demás modelos).
- Felicitaciones! Ya tienen todas las entregas necesarias para regularizar aprobadas (sin contar la de promoción), y están dentro de la primer tanda de grupos en lograrlo :) Igualmente les recomendaría arreglar lo que mencioné de los archivos en el repo, ya que queda bastante mal si en algún momento quieren mostrar el proyecto en algún laburo o entrevista.
Grupo 3 (Lectoescritores)
| Entrega | Estado en la primer fecha | Estado en la segunda fecha |
|---|---|---|
| 1 | No aprobada, correcciones pendientes | Aprobada |
| 2 | No aprobada, detalles a corregir | Aprobada |
| 3 | Aprobada, pero un detalle a corregir | |
| 4 | No aprobada, detalles a corregir | Aprobada |
| 5 | No aprobada, faltan items | Aprobada |
| 6 | Aprobada |
Correcciones pendientes:
- [E6] El settings hace que cuando se corran las cosas en el container de docker, la url de todo lo que es media esté rota (porque setea incorrectamente el MEDIA_URL, que es una url web, no un path a archivos). Como es un detalle la entrega queda aprobada, pero estaría bueno que lo dejen arreglado.
Otras observaciones:
- La review de sitios anteriores se hizo casi principalmente en base a diseño, hubiera estado bueno considerar más de la funcionalidad.
- Cómo sabrían si alguien leyó un escrito? Visitar la página es suficiente? Tiempo? Y si simplemente dejé la pestaña abierta? Etc.
- A la cuenta del repo en github le faltan más patillas para poder llevar ese nombre :)
- En los escenario de uso de usuarios reviewer y moderador, pasa lo que pasaba con el del usuario lector antes: no es tan cárlo cómo hacen las cosas que dice que van a hacer (ej: el moderador, dónde ve esas cosas? cómo banea? etc). Igualmente se los consideramos aprobados, pero como consejo, es buena idea tratar de pensar esas cosas para saber qué es el trabajo que hay que hacer en el sitio (ej: darse cuenta de que hace falta un panel de moderación, etc).
- Genial que implementaron login con la cuenta de Google!
- Quién es el usuario "unidad_de_simulacion_militar_latina"???
- Me esmeré en uno de los comentarios.
- Felicitaciones! Ya tienen todas las entregas necesarias para regularizar aprobadas (sin contar la de promoción) :)
Grupo 4 (Recetas)
| Entrega | Estado en la primer fecha | Estado en la segunda fecha | Estado en extensión extraordinaria (solo 1) |
|---|---|---|---|
| 1 | No aprobada, correcciones pendientes | Aprobada | |
| 2 | No aprobada, detalles a corregir | Aprobada | |
| 3 | No aprobada, correcciones y consignas pendientes | Aprobada | |
| 4 | No aprobada, detalles a corregir | Aprobada | |
| 5 | No aprobada, faltan items | No aprobada, faltan items | Aprobada |
| 6 | No presentada (no es obligatoria) |
Correcciones pendientes:
Ninguna.
Otras observaciones:
- En el escenario de uso del moderador, no es tan cárlo cómo hacen las cosas que dice que van a hacer (ej: el moderador, dónde ve esas cosas? cómo banea? etc). Igualmente se lo consideramos aprobado, pero como consejo, es buena idea tratar de pensar esas cosas para saber qué es el trabajo que hay que hacer en el sitio (ej: darse cuenta de que hace falta un panel de moderación, etc).
- Espero que les guste la receta familiar que subí (usuario foobar2). Es un secreto guardado por generaciones (dos generaciones, mi papá y yo. Y tampoco tan secreto porque le contamos a gente)
- Muy buena la decisión de customizar así el admin!
- En la búsqueda de texto, sería super útil incluir otros campos más que solo el título de la receta. Recuerden que buena parte de las ventajas de los sistemas de búsqueda están en cómo pueden distuinguir qué cosas son relevantes en un documento, usar la cantidad de apariciones, etc.
Grupo 5 (Salas de chat)
| Entrega | Estado en la primer fecha | Estado en la segunda fecha |
|---|---|---|
| 1 | No aprobada, correcciones pendientes | Aprobada |
| 2 | Aprobada | |
| 3 | No aprobada, detalles a corregir | Aprobada |
| 4 | No aprobada, correcciones pendientes | Aprobada |
| 5 | No aprobada, faltan items | Aprobada |
| 6 | No aprobada | Aprobada |
Correcciones pendientes:
Ninguna.
Otras observaciones:
- No hacía falta poner dirección en los perfiles de usuario, porque es algo que no afecta a cómo ese usuario va a usar el sitio :)
- Gracias por tomarse el trabajo de hacer un índice :)
- Un tema importante a pensar sería cómo evitar el spam, gente que se una a cientos de salas a postear publicidad (incluso bots), creando cuentas falsas cuando le borran sus cuentas, etc.
- Reddit tiene la posibilidad de administrar/moderar los subreddits (el creador puede hacerlo, y puede delegarlo también).
- Felicitaciones! primer grupo del 2020 que aprueba una entrega al primer intento! :)
- Felicitaciones por tener el repo más libre de archivos incorrectos del 2020 :) (como la db, archivos pyc, etc).
- Sería útil proveer algunos defaults razonables en los campos de límites de mensajes y duración de la sala, cosa de que el usuario no los tenga que "pensar" si no quiere. O incluso dejarlos vacíos para permitir crear una sala sin limitaciones ni duración.
- En la indexación de las salas, podría ser útil incluir los nombres de los tags que tiene, para que si el usuario busca con una palabra que es un tag, le encuentre también salas tagueadas con ese tag.
- Felicitaciones! Ya tienen todas las entregas necesarias para regularizar aprobadas (sin contar la de promoción), y están dentro de la primer tanda de grupos en lograrlo :)
Grupo 6 (Campings)
| Entrega | Estado en la primer fecha | Estado en la segunda fecha |
|---|---|---|
| 1 | No aprobada, correcciones pendientes | Aprobada |
| 2 | No aprobada, detalles a corregir | Aprobada |
| 3 | No aprobada, correcciones pendientes | Aprobada |
| 4 | No aprobada, detalles a corregir | Aprobada, con un solo detalle a corregir |
| 5 | No aprobada, faltan items | Aprobada |
| 6 | No aprobada | Aprobada |
Correcciones pendientes:
Ninguna.
Otras observaciones:
- Genial que ya están indagando en cómo darle estilo :)
- Bien ahí usando messages para el mensaje del mail de activación de la cuenta, recuérdenme comentarle al resto de los grupos.
- Me llevó tiempo, pero recién ahora caigo el por qué del nombre, es genial :D
- http://vivisencarpa.herokuapp.com/foro/tema/2/ (hilo completo)
- Genial que hayan hecho búsqueda por más de un tipo de contenido, fueron el único grupo que se animó a hacer más de lo mínimo indispensable en ese aspecto :) Un detalle mejorable: por lo general es buena idea dejar la barra de búsqueda en la página de resultados también (y sin borrarle el texto ingresado), para que el usuario no tenga que volver a atrás cuando quiere cambiar un poco lo que buscó, ni tenga que tipearlo de vuelta de cero.
- Consejo: no se pedía en la consigna por lo que no hace falta que lo hagan, pero sería útil que la data de MEDIA también esté en /data al utilizar docker. Fue algo extra que muchos otros grupos hicieron, y por eso se los comento como cosa interesante.
Grupo 7 (Comunidades educativas)
| Entrega | Estado en la primer fecha | Estado en la segunda fecha | Estado en extensión extraordinaria (solo 1) |
|---|---|---|---|
| 1 | No aprobada, correcciones pendientes | Aprobada | |
| 2 | No aprobada, faltan items | Aprobada | |
| 3 | No aprobada, correcciones y consignas pendientes | No aprobada, correcciones pendientes | Aprobada |
| 4 | No aprobada, correcciones pendientes | Aprobada | |
| 5 | No aprobada, faltan items | Aprobada | |
| 6 | Aprobada |
Correcciones pendientes:
- [E6] El settings hace que cuando se corran las cosas en el container de docker, la url de todo lo que es media esté rota (porque setea incorrectamente el MEDIA_URL, que es una url web, no un path a archivos). Como es un detalle la entrega queda aprobada, pero estaría bueno que lo dejen arreglado.
Observaciones
- Cuando hacen cambios como el reciente en el guardado de la info de contacto, siempre asegúrense de que la data existente sigue funcionando. Renegué bastante con errores usando todos los usuarios que tenía antes, hasta entender que era por ese cambio, y que si creaba usuarios nuevos sí funcionaban :)
- Felicitaciones! Ya tienen todas las entregas necesarias para regularizar aprobadas (sin contar la de promoción) :)