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

Feature/saldo cuentas #9

Merged
merged 6 commits into from
Mar 16, 2022
Merged

Feature/saldo cuentas #9

merged 6 commits into from
Mar 16, 2022

Conversation

salcedogeiner
Copy link
Contributor

Copy link
Contributor

@AlexFBP AlexFBP left a 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

controllers/saldo.go Show resolved Hide resolved
@@ -0,0 +1,32 @@
CREATE TABLE movimientos_contables.saldo (
Copy link
Contributor

@AlexFBP AlexFBP Mar 16, 2022

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:

  1. abrirlo con Inkscape
  2. darle dobleclick de tal manera que quede todo el esquema seleccionado
  3. Menu "editar" > Opcion "Ajustar la pagina a la seleccion" (Ctrl+Mayus/Shift+R)
  4. Guardar cambios (y ahí si hacer el commit del archivo svg)

Copy link
Contributor Author

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

database/scripts/20220309_055816_crear_tabla_saldo_up.sql Outdated Show resolved Hide resolved
database/scripts/20220309_055816_crear_tabla_saldo_up.sql Outdated Show resolved Hide resolved
models/movimiento.go Outdated Show resolved Hide resolved
models/saldo.go Outdated Show resolved Hide resolved
@AlexFBP AlexFBP merged commit 3e1d5d3 into develop Mar 16, 2022
@AlexFBP AlexFBP deleted the feature/saldo_cuentas branch March 16, 2022 19:43
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.

2 participants