-
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
Release/1.0.0 --> master #173
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ilizar consulta en el cliente de presupuesto #425
feat: Se crea Endpoint Para construir arbol rubro apropiacion para ag…
…cion feat: Actualizar swagger con ArbolRubroApropiacion
- Permitir que los check de commit fallen, queda a criterio del lider la aprobación de los PR. Es, a efectos prácticos, mas importante que compile el código - Permitir que los test fallen, con el fin de implementar TDD, Test-Driven Development, en que si los test se hacen bien, desde el primer momento deberían fallar todos y con esto el objetivo se convierte en hacer que los test empiecen a ser exitosos - No disparar builds cuando se haga `git push` de tags - Generar build también para ramas hotfix/* y feature/*, con el fin de saber de antemano que se va a mandar a release/* código que al menos compiló
- PLAN_CUENTAS_CRUD ya no está en servicio, las peticiones deberán eliminarse o retornar HTTP 424 - Agregado TODO.md para listar las dependencias anteriores - Marcados helpers como deprecated, segun https://go.dev/blog/godoc
Se crearon 3 helper (para crear movimientos proceso externo, para consultar el ultimo movimiento y crear movimientos detalle), un interceptor el cual evalua la necesidad la actualiza si no es para aprobar, pero si es, evalua los montos si hay presupuesto, si no hay no permite el cambio, si si hay, hace el movimiento y actuliza la necesidad.
Se realizo ajustes en el codigo de los controladores par que la informacion que se envia desde el Mid, llegue de manera corracta para el uso en CRUD de movimiento, se agregaron algunos casos de control de errores, se actualizaron los modelos de para estos funciones segun versiones del CRUD
…cesidad Feature/get full necesidad
doc: release/0.1.0
release/1.0.0
…cionUtils Feature/actualizacion utils
…osNecesidades Feature/movimientos necesidades
Develop to Release
…osNecesidades fix: generar consecutivo
Develop to Release
…osNecesidades fix: Ajuste para ignorar monto parcial 0
Develop to Release
Feature/defer crpcdp
Develop to Release
AlexFBP
commented
Jun 19, 2022
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.
Primer release, VoBo en #185 y anteriores.
Pendiente por revisión de Arquitectura para el primer lanzamiento a producción
Aparentemente ya había sido desplegado en el pasado el ambiente de pruebas, solo quedaría pendiente las pruebas de carga y seguridad |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.