-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/saldo cuentas #9
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En general bien. Algunos comentarios respecto a los lineamientos de control de errores y propiedades de valores monetarios
Respecto al PGmodeler, se puede actualizar, bien sea en este PR (mejor 🙏 ) o directamente después de crear el siguiente release, agregando un commit más a la rama release
@@ -0,0 +1,32 @@ | |||
CREATE TABLE movimientos_contables.saldo ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Falta actualizar los modelos de PgModeler (dbm y svg) (en la carpeta database)
Si tienes la DB montada en local, se puede generar el pgmodeler a partir de la DB. Se recomienda ordenar las entidades de tal manera que las relaciones se vean el 1
a la izquierda y el n
a la derecha
Una vez ajustado el pgmodeler y generado el SVG, dado que a veces se ve mal al intentar abrirlo desde un navegador, se recomienda ajustar el SVG con Inkscape para que al abrirlo, sobretodo desde el navegador, se vea bien, no se corte
El ajuste es simplemente:
- abrirlo con Inkscape
- darle dobleclick de tal manera que quede todo el esquema seleccionado
- Menu "editar" > Opcion "Ajustar la pagina a la seleccion" (Ctrl+Mayus/Shift+R)
- Guardar cambios (y ahí si hacer el commit del archivo svg)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
queda pendiente actualizacion del modelo
Co-authored-by: Alex Bustos <[email protected]>
Co-authored-by: Alex Bustos <[email protected]>
udistrital/contabilidad_cliente#144