la-velada-web-oficial
la-velada-web-oficial copied to clipboard
refactor: update toast component :recycle:
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.