Correcciones2022 - ucseiw-team/catedra GitHub Wiki

Grupo 1: (artesanos) https://rafaela-emprende.herokuapp.com/

  • Entrega 0 aprobada
  • Entrega 1 aprobada
  • Entrega 2 aprobada
  • Entrega 3 aprobada
  • Entrega 4 aprobada
  • Entrega 5 aprobada
  • Entrega 6 aprobada

Correcciones pendientes:

Comentarios:

  • 50 años? No parece! :D
  • Traten de mantener todo en un solo idioma. (el filtro de categorías dice "Category")
  • Probablemente no hace falta tener una tabla/modelo aparte para Puntaje. Con que sea un valor numérico en EmprendedorPuntaje hubiera sido suficiente, y si está limitado a un set de valores fijos, se podría usar un campo con opciones directamente. Porque seguro no es algo que vayan a cambiar en la db sin cambiar junto el código, así que no suma nada que sea una tabla, y sí agrega complejidad innecesaria.
  • Quizás se podría trabajar un poco más el tema del contraste en los colores elegidos para textos y sus fondos.
  • En general, felicitaciones por el nivel de terminado que le dieron a todas las features.
  • Muy bien lograda la funcionalidad de búsqueda.

Grupo 2: (lavaderos) https://lavaderos-online.herokuapp.com/

  • Entrega 0 aprobada
  • Entrega 1 aprobada
  • Entrega 2 aprobada
  • Entrega 3 aprobada
  • Entrega 4 aprobada
  • Entrega 5 aprobada
  • Entrega 6 aprobada

Correcciones pendientes:

Comentarios:

  • Muy bien pensado el cambio de idea respecto a turnos y demás.
  • Es super discutible lo de que los usuarios no van a poder hacer nada que sea inapropiado. Por ejemplo: si me creo un usuario para todo el tiempo molestar a los lavaderos reservándoles el lugar para lavado inmediato y no voy? O si me creo un lavadero falso solamente para hacer mala publicidad de uno real? O si creo un lavadero con una foto de alguien para molestarlo? Suena a que realmente sí es necesario un estado de baneo. No subestimen la capacidad de la gente para hacer daño :)
  • Muy buenos los mails en general, bien cuidado el estilo y demás.
  • En el mundo real probablemente sea mejor mostrar notificaciones en la app para los pedidos de lavados y sus respuestas, en vez de necesitar revisar el mail.

Grupo 3: (aprender inglés) https://englishwebsitegroup3.herokuapp.com/

  • Entrega 0 aprobada
  • Entrega 1 aprobada
  • Entrega 2 aprobada
  • Entrega 3 con correcciones pendientes
  • Entrega 4 aprobada
  • Entrega 5 aprobada
  • Entrega 6 (para regularizar) aprobada

Correcciones pendientes:

  • Están usando un template base bien implementado en algunas páginas (detalle de ejercicios, listas de ejercicios, aunque ahí por ejemplo podría usarse un solo template con variables para lo que cambia). Pero en otras páginas siguen sin usar el base y tienen en vez copiados y pegados fragmentos del mismo (ej: crear ejercicio), y en otras directamente tienen algo que nada que ver al base (ej: la sección de profesores, la home). En algún caso se puede justificar que alguna parte se vea diferente (ej: en algunos casos la home no logueado es tan diferente del resto del sitio que puede no usar el base), pero en el caso de ustedes, están haciendo mucha mezcla de estilos de navegación, y mucho copy-paste de código cuando eso se tiene que hacer reutilizando código. Usen un base para todas las páginas, salvo alguna página que quieran justificar que realmente tiene sentido que sea completamente diferente al resto del sitio.

Comentarios:

  • En los escenarios de uso todavía la cosa está muy genérica, lo que no los ayuda a ustedes mismos al no quedar claro bien cómo van a resolver las necesidades de los usuarios. Sería importante si pueden, que lo acomoden para que sea más específico.
  • Ningún usuario va a acceder desde el celular? Por ejemplo, la gente que tiene horas de viaje al trabajo, no podría aprovecharlas para practicar inglés con el celu durante el viaje?
  • Es interesante la opción de usar Disqus, pero el tema es que la entrega estaba pensada para que ustedes programen esas features (de contacto y notificaciones) y aprendan programándolas. Lo vamos a tomar igual porque no fuimos específicos en la consigna, pero a futuro lo vamos a aclarar más.
  • En el robots.txt sería bueno excluir /admin

Grupo 4: (e-sports) https://torneosgod.herokuapp.com/ PROYECTO CANCELADO

Grupo 5: (música) (url removida por problemas de seguridad) PROYECTO CANCELADO

Grupo 6: (barrios) https://noti-barrios.herokuapp.com/

  • Entrega 0 aprobada
  • Entrega 1 aprobada
  • Entrega 2 aprobada
  • Entrega 3 aprobada
  • Entrega 4 aprobada
  • Entrega 5 aprobada
  • Entrega 6 (para regularizar) aprobada

Correcciones pendientes:

Comentarios:

  • Las noticias permiten subir fotos, pero luego no se ven en ninguna parte?
  • Traten de mantener todo en un solo idioma.
  • Bien usado el tema del grupo de moderadores para permitir o no aceptar gente.

Grupo 7: (carpooling) https://viajando-ando-heroku.herokuapp.com/

  • Entrega 0 aprobada
  • Entrega 1 aprobada
  • Entrega 2 aprobada
  • Entrega 3 aprobada
  • Entrega 4 aprobada
  • Entrega 5 aprobada
  • Entrega 6 aprobada

Correcciones pendientes:

Comentarios:

  • En los escenarios de uso todavía la cosa está muy genérica, lo que no los ayuda a ustedes mismos al no quedar claro bien cómo van a resolver las necesidades de los usuarios. Sería importante si pueden, que lo acomoden para que sea más específico.
  • Se puede crear más de un usuario con el mismo mail, eso puede ser peligroso (fácil de abusar, y fácil de que aparezcan bugs relacionados a cosas como envío de mails con notificaciones, respuestas, etc).
  • Luego de solicitar unirse a un viaje, sería bueno redireccionar al detalle de ese viaje, en vez de a la búsqueda con todos los campos reseteados. O al menos a la búsqueda pero sin resetear los filtros, para permitir ir viendo y solicitando a varios viajes seguidos.