Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tarea #1275 - agregar modelo LogAudit #1626

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

elguitarraverde
Copy link
Contributor

Descripción

  • Problema

Ahora mismo cuando se crea una factura, se añade al log del canal audit un mensaje indicando quién ha modificado la factura y los datos de contexto. El problema viene cuando por algún motivo no se llega a guardar la factura (por ejemplo si no hay stock) y se deshace la transacción. La factura no se crea, pero el mensaje si se añade al canal audit.

  • Solución

Crear un modelo LogAudit para guardar directamente en base de datos estos datos de auditoría. De esta forma, si se rechaza la transacción, se rechazan también los datos de auditoría.

¿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