Skip to content

fix: autocomplete bug on error#1968

Open
abdedarghal111 wants to merge 1 commit into
NeoRazorX:masterfrom
abdedarghal111:tarea3386
Open

fix: autocomplete bug on error#1968
abdedarghal111 wants to merge 1 commit into
NeoRazorX:masterfrom
abdedarghal111:tarea3386

Conversation

@abdedarghal111
Copy link
Copy Markdown
Contributor

Descripción

https://facturascripts.com/roadmap/3386

  • Se ha modificado el widget autocomplete para que suelte un error apropiado para el cliente en vez de soltar un error cualquiera. Los diferentes handlers implementados son:

    • Informar si hay un fallo en la comunicación (perdida de conexión)
    • Si hay un error 500 informar de ello
    • Si hay un error 400 avisa apropiadamente
    • Si hay otro error también avisa
    • Si se le pasa al cliente un key o value inválidos, suelta una warn por consola para que el desarrollador lo tenga en cuenta.
  • Lo que se ve que son cambios relacionados con I18N simplemente es que en el .twig se agrega como variable global los mensajes de error y luego se rescatan en la función de javascript. De esta manera los errores pueden verse en los diferentes idiomas para el cliente.

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

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.

1 participant