Skip to content

Conversation

@molro
Copy link
Owner

@molro molro commented Feb 23, 2022

Entrega Final

@StratocasterO
Copy link

No aconsegueixo fer-ho funcionar, equip! Em penso que us heu deixat alguna instrucció pel camí o alguna cosa així 🤔

@molro
Copy link
Owner Author

molro commented Mar 31, 2022

Omar hemos corregido la app.

Saludos

@molro molro changed the title Update README.md Entrega DevTeams Corregida Mar 31, 2022
@StratocasterO
Copy link

Buen trabajo, equipo! Ahora funciona, pero hay cosas mejorables:

  • Estaría bien poner un selector de fechas, para que no puedan pasar cosas como que la fecha de finalización sea anterior a la de inicio ❗
  • Habría que uniformizar cuando enseñáis las cosas y cuando no: p.e. antes de eliminar las muestra para que veas los ID, pero al seleccionar para ver una no
  • Estaría bien también mostrarlas diferentes en función del estado (colores?)
  • Por alguna razón, si elimino una que no es la última los IDs cambian para rellenar el hueco. Esto puede confundir al usuario ❓
  • Cuando no hay resultados muestra undefined, sería mucho mejor decirle al usuario que no hay resultados con un mensaje

Bien hecho! Ahora está correcto como mínimo 😉

molro added 2 commits October 27, 2022 12:55
Updating profile name and URL to GitHub Repo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants