udistrital / financiera_documentacion

Repositorio para contener la documentación y diseños relacionados con el sistema de gestión financiera Kronos de la Universidad Distrital.
GNU General Public License v3.0
1 stars 2 forks source link

Definición API para consumir cuentas bancarias (de tesorería) en plan de cuentas (contabilidad) #363

Open AlexFBP opened 3 years ago

AlexFBP commented 3 years ago

Consideraciones

Parte 1 - API

Fecha de entrega estimada: 21/09/21 Duración Estimada: 1 día

Parte 2 - Desarrollo

(D): @diagutierrezro - (J): @Jeisson005

Jeisson005 commented 3 years ago

En reuniones realizadas con @diagutierrezro: Se encuentra que la API giros_crud ya cuenta con un endpoint que permite obtener cuentas bancarias, por lo que se decide que esta será la interfaz común de donde se consumirán las cuentas bancarias para los clientes de tesorería y de contabilidad.

Modelo de la entidad: image

Endpoint que se consume: https://autenticacion.portaloas.udistrital.edu.co/apioas/giros_crud/v1/cuenta_bancaria

Resultado obtenido:

{
    "Data": [
        {
            "Id": 1,
            "NombreId": 1,
            "NumeroCuenta": 123456789,
            "BancoId": 1,
            "TipoCuenta": 1,
            "Activo": true,
            "FechaCreacion": "2021-09-24 21:49:47.944609 +0000 +0000",
            "FechaModificacion": "2021-09-24 21:49:47.951662 +0000 +0000"
        }
    ],
    "Message": "Request successful",
    "Status": "200",
    "Success": true
}

Se considera agregar un campo para enlazar una sucursal bancaria, sin embargo en reunión con @mcrubianot se confirma que el usuario no requiere las sucursales por lo que no se modifica el modelo actual.

La API actual ya cuenta con todos los lineamientos de la OAS, por lo que ya se cuentan con los parámetros query, fields, sortby, order, limit, offset así como la validación de datos entrantes y control de posibles errores.

AlexFBP commented 3 years ago

Propuesta, que en alguno de los MID se cree un controlador, que retorne lo siguiente:

[
  {
    "IdCuenta": 1,
    "Banco": "Banco de Occidente", // Podría ser todo el tercero
    "NumeroCuenta": "32423423432"
  },
  {
    "IdCuenta": 1,
    "Banco": "Banco de Bogotá", // Podría ser todo el tercero
    "NumeroCuenta": "32423423432"
  }
]

Posiblemente de Tesorería

Del lado de tesorería, sí que vale la pena mostrar un select con los bancos.

En cualquier caso, dejar claro (al menos en tesorería, o mejor aún, en la documentación Swagger)

diagutierrezro commented 3 years ago

Finalmente el usuario decide que si es necesaria la información de la sucursal bancaria por lo que se define hacerlo de la siguiente manera:

diagutierrezro commented 3 years ago

Se realiza la migración en el crud para modificar el campo mencionado (sucursal_id), tambien se realiza el cambio en el modelo para que coincida con el cambio en la db. Esto se evidencia en los siguientes commits 1 y 2

Jeisson005 commented 3 years ago

Se creo el controlador cuenta_bancaria_banco en la API https://github.com/udistrital/tesoreria_mid

diagutierrezro commented 3 years ago

Se crea la vista completa de sucursales:

Jeisson005 commented 3 years ago

Se agrega el campo CuentaBancariaID al API cuentas_contables_crud

Jeisson005 commented 3 years ago

Se modifica la vista en contabilidad_cliente, el resultado es el siguiente: image image