la-velada-web-oficial icon indicating copy to clipboard operation
la-velada-web-oficial copied to clipboard

refactor: update toast component :recycle:

Open Jesus-Rojas opened this issue 10 months ago • 6 comments

Descripción

Existe demasiado condicional en la funcion toast, la mayoria de casos valida si existe o no cierto toastOption. Tambien he visto demasiados strings mágicos.

Cambios propuestos

  • Todos los campos son opcionales, la mejor opción para evitar tanto condicional es crear parámetros por defecto.
  • Actualice el componente toast para usar enums en vez de strings mágicos, esto incrementa la mantenibilidad del componente toast y se evitan bugs innecesarios.

Comprobación de cambios

  • [x] He revisado que no haya ninguna PR (pull request) ya abierta con un problema similar, siguiendo el apartado de buenas prácticas
  • [x] He revisado localmente los cambios para asegurarme de que no haya errores ni problemas.

Impacto potencial

En cuanto a UX no afecta, en cuanto a mantenibilidad si es necesario el cambio.

Jesus-Rojas avatar Apr 24 '24 20:04 Jesus-Rojas