Conversation
negro89
force-pushed
the
HUDS-184
branch
4 times, most recently
from
71d5e3e to
7f7d161
Compare
negro89
force-pushed
the
HUDS-184
branch
2 times, most recently
from
d5764a5 to
74d4a46
Compare
|
Error exportando la huds , el archivo zip tiene el contenido y pesa 500 kb pero al momento de descargar esta con error el pdf con 0 bytes
|
|
Se volvio a probar luego del nuevo commit :
|
negro89
left a comment
negro89
left a comment
There was a problem hiding this comment.
Nico, la idea general esta bien pero dejo algunas observaciones en base al ultimo commit
|
|
||
| // ruta para verificar si existe historial antes de generar el zip | ||
| ExportHudsRouter.post('/export/check-history', async (req: any, res, next) => { | ||
| const { checkHistory } = require('./exportHuds.controller'); |
There was a problem hiding this comment.
este tipo de importaciones no solemos hacerlas; por convencion preferimos importaciones "estaticas" al principio del archivo. Ademas el uso de require imports está desaconsejado desde angular 10 ya que penaliza la optimización 😅
| } | ||
|
|
||
| // Verifica si hay historial antes de generar el zip | ||
| export async function checkHistory(params) { |
There was a problem hiding this comment.
es por acá el camino, pero hay mucho codigo repetido con el createFile. Lo ideal sería modularizar el bloque de busqueda de createFile para poder llamarlo desde ambos métodos.
Tener en cuenta el comentario de la linea 78, si no se hace de esa forma no va a levantar nunca un cda.
| query['solicitud.organizacion.id'] = organizacion; | ||
| } | ||
|
|
||
| const countPrestaciones = await Prestacion.countDocuments(query); |
There was a problem hiding this comment.
la idea era usar findOne o exists que son mas eficientes ya que que tanto el coundDocument como el count primero recorren toda la coleccion buscando el total de los resultados y despues encima realizan el conteo. Mientras los primeros apenas encuentran un match cortan la ejecución y retornan el documento o un bool respectivamente.
2 participants
Requerimiento
https://proyectos.andes.gob.ar/browse/HUDS-184
Funcionalidad desarrollada
IMPORTANTE: Es importante probar todas las pantallas desde donde se descargan pdfs y envian mails. Dejar comentado aquellos que haya que ajustar 🙏🏼
UserStories llegó a completarse
Requiere actualizaciones en la base de datos
GLPI RELACIONADO: https://glpi.andes.gob.ar/front/ticket.form.php?id=5081