Feature/cuenta bancaria #129
Conversation
|
|
||
| // Terceros | ||
|
|
||
| public getCuentasBancarias(parameters?: { id?: any, query?: any, fields?: string[], sortby?: string[], order?: string[], limit?: number, offset?: number }) { |
There was a problem hiding this comment.
Se observa que { id?: any, query?: any, fields?: string[], sortby?: string[], order?: string[], limit?: number, offset?: number } se repite en varias partes.
Guardar esta estructura como un nuevo modelo de datos, por ejemplo, algo como BeegoQueryParams:
interface BeegoQueryParams {
id?: string; // evitar el any
query?: string; // evitar el any
fields?: string[];
sortby?: string[];
order?: string[];
limit?: number;
offset?: number;
}
...
export {..., BeegoQueryParams};De tal manera que el getCuentasBancarias (y dondesea que se use esta misma estructura) se pueda reescribir como
import {BeegoQueryParams} from '...';
...
public getCuentasBancarias(parameters?: BeegoQueryParams){...}(Si no funciona con interface usar simplemente class)
|
|
||
| // Tercero | ||
|
|
||
| public getTerceros(parameters?: { id?: any, query?: any, fields?: string[], sortby?: string[], order?: string[], limit?: number, offset?: number }) { |
There was a problem hiding this comment.
Ver lo comentado en src/app/@core/helpers/giros/girosHelper.ts
| } | ||
|
|
||
| // Tercero_tipo_tercero | ||
| public getTerceroTipoTercero(parameters?: { id?: any, query?: any, fields?: string[], sortby?: string[], order?: string[], limit?: number, offset?: number }) { |
There was a problem hiding this comment.
Ver lo comentado en src/app/@core/helpers/giros/girosHelper.ts
|
|
||
| // tesoreria_mid | ||
|
|
||
| public getCuentasBancarias(parameters?: { id?: any, limit?: number, offset?: number }) { |
There was a problem hiding this comment.
Posiblemente se pueda hacer algo similar a lo comentado en src/app/@core/helpers/giros/girosHelper.ts
| * @param offset Start position of result set. Must be an integer | ||
| * @returns | ||
| */ | ||
| getv2(endpoint: string, parameters?: { id?: any, query?: any, fields?: string[], sortby?: string[], order?: string[], limit?: number, offset?: number }) { |
There was a problem hiding this comment.
Ver lo comentado en src/app/@core/helpers/giros/girosHelper.ts respecto a parameters
| @@ -13,6 +13,7 @@ export const environment = { | |||
| WSO2_SERVICE: 'https://autenticacion.portaloas.udistrital.edu.co/apioas/', | |||
| PLAN_CUENTAS_MONGO_SERVICE: 'https://autenticacion.portaloas.udistrital.edu.co/apioas/plan_cuentas_mongo_crud/v1/', | |||
| CUENTAS_CONTABLES_SERVICE: 'https://autenticacion.portaloas.udistrital.edu.co/apioas/cuentas_contables_crud/v1/', | |||
| TESORERIA_MID_SERVICE: 'http://pruebasapi2.intranetoas.udistrital.edu.co:8214/v1/', | |||
There was a problem hiding this comment.
- Esto solo funcionaría bajo VPN, no se encuentra desplegado?
- Ajustar no solo acá sino en el .test.ts y en el .prod.ts
| @@ -14,6 +14,7 @@ export const environment = { | |||
| PLAN_CUENTAS_MONGO_SERVICE: 'https://autenticacion.portaloas.udistrital.edu.co/apioas/plan_cuentas_mongo_crud/v1/', | |||
| CUENTAS_CONTABLES_SERVICE: 'https://autenticacion.portaloas.udistrital.edu.co/apioas/cuentas_contables_crud/v1/', | |||
| CONFIGURACION_SERVICE: 'https://autenticacion.portaloas.udistrital.edu.co/apioas/configuracion_crud_api/v1/', | |||
| TESORERIA_MID_SERVICE: 'http://pruebasapi2.intranetoas.udistrital.edu.co:8214/v1/', | |||
There was a problem hiding this comment.
Revisar lo comentado en src/environments/environment.ts
|
se cierra PR ya que se continua desarrollo en otra rama |
Antes de realizar el pull, se debe realizar el merge a las dependencias.
Todo se probó de forma local con las configuraciones adecuadas en las ramas correspondientes a esos pr
Depende de PRs:
udistrital/cuentas_contables_crud#56
udistrital/tesoreria_mid/pull/19
Close #92
Close udistrital/financiera_documentacion#363