[ADD] Search for tax according to the CAByS code of the product and Expiration date of the Ministerio de Hacienda certificate #209
Conversation
[UPDATE]
…lidate the CAByS code
norubo
changed the title
¿Podría indicarme que decía el log del servidor? Para verificar cual pudo ser la causa y corregirlo |
Claro men te lo adjunto, pura vida. |
|
Un comentario y una pregunta: Debería utilizar ORM para hacer el update y no hacerlo directo con SQL Query. ¿Porqué puso que se actualizara la fecha al cambiar el mobile? |
|
@lmoram El error se genera cuando se cargan las vistas y los campos nuevos del modelo no están cargados, cuando lo instalo desde 0 no da problemas, sin embargo cuando ya tengo una bd con la versión anterior del módulo me da error, lo que hice en ese caso fue comentar los campos nuevos que creé y actualicé, luego los descomenté y volví a actualizar y funcionó correctamente, esas es una solución temporal @WhiteTiger08 respecto al ORM y el Query yo prefiero realizarlo con ORM, lo útlimo creo que no me quedó claro, lo de la fecha del certificado ahorita es la primer versión y se actualiza al "consultar el token" desde la info de la compañía, en el futuro debe realizarse con una acción planificada y generar una notificación del vencimiento |
There was a problem hiding this comment.
@lmoram El error se genera cuando se cargan las vistas y los campos nuevos del modelo no están cargados, cuando lo instalo desde 0 no da problemas, sin embargo cuando ya tengo una bd con la versión anterior del módulo me da error, lo que hice en ese caso fue comentar los campos nuevos que creé y actualicé, luego los descomenté y volví a actualizar y funcionó correctamente, esas es una solución temporal
@WhiteTiger08 respecto al ORM y el Query yo prefiero realizarlo con ORM, lo útlimo creo que no me quedó claro, lo de la fecha del certificado ahorita es la primer versión y se actualiza al "consultar el token" desde la info de la compañía, en el futuro debe realizarse con una acción planificada y generar una notificación del vencimiento
Perdón, al revisar el código estaba collapsado y parecía que con el mobile cambiaba la fecha de expiración. Lo vi mal por revisarlo desde el celular.
| peticion = requests.get(end_point, headers=headers, timeout=10) | ||
|
|
||
| ultimo_mensaje = 'Fecha/Hora: ' + str(datetime.now()) + ', Codigo: ' + str(peticion.status_code) + ', Mensaje: ' + str(peticion._content.decode()) | ||
| self.env.cr.execute("UPDATE res_company SET ultima_respuesta_cabys='%s' WHERE id=%s" % (ultimo_mensaje, self.company_id.id)) |
There was a problem hiding this comment.
Esto debería ser con ORM
There was a problem hiding this comment.
Voy a corregirlo, me basé en lo de la información de la cédula, por ende utiliza eso mismo del SQL, muchas gracias
Description of the issue/feature this PR addresses:
1 - Validate the tax corresponding to the CAByS code
2 - Manage the expiration date of the Ministerio de Hacienda certificate
Current behavior before PR:
1 - There is only the field to control the CAByS code but it does not indicate the tax
2 - The expiration date is not being controlled
Desired behavior after PR is merged:
1 - Show the tax on the product based on the CAByS code
2 - Now in the company information it shows the expiration date of the cryptographic key