Correccion - Mejora #5033
Actualizado por Oscar Abad hace alrededor de 1 año
h1. Especificación Incluir tareas para la notificación a los usuarios involucrados de aquellos acuerdos que siguen abiertos pasada la fecha de finalización. Según conversación con Sito, se trata de una notificación que el departamento de Jurídico ha demandado. Para no generar demasiados correos con acuerdos viejos, las notificaciones deberían generarse diariamente, mandando un correo si es un mes antes, una semana antes o el mismo día en que finaliza la vigencia del proyecto (acuerdo_datos.fecha_fin). h1. Pruebas Ya implementado en gor/lib/tareas.rb <pre> # Notificacion de acuerdos abiertos con fecha de finalizacion superada def self.notifica_acuerdos_abiertos Rails.logger.info "*** Notifica acuerdos abiertos (#{I18n.l(Time.now)})" acuerdos = Acuerdo.expirados_y_abiertos num_acuerdos = acuerdos.count if num_acuerdos > 0 Rails.logger.info "*** Hay #{num_acuerdos} acuerdos abiertos que han finalizado" acuerdos.each do |acuerdo| fecha_finalizacion = acuerdo.ultimos_datos.fecha_fin acuerdo.usuarios_notificacion.each do |destinatario| Rails.logger.info "[Tareas.notifica_acuerdos_abiertos] Enviando notificación de acuerdo expirado (#{acuerdo.id}) a '#{destinatario.correoe}'" begin AcuerdosMailer.acuerdo_expirado_abierto(destinatario, acuerdo).deliver_later rescue => e Rails.logger.error "[Tareas.notifica_acuerdos_abiertos] No se ha podido mandar el mail a '#{destinatario.correoe}': #{e.inspect}" end end end end end </pre> Ejecutándose cada semana en plugins/oei_erp/lib/clock.rb <pre> # Envia notificaciones de acuerdos abiertos todos los domingos a las 10.45h every(1.week, 'Convenios abiertos', at: 'Sunday 10:45') do |job| Tareas.notifica_acuerdos_abiertos end </pre> Según los datos de la BD anonimizada de producción: <pre> *** Notifica acuerdos abiertos (03/10/2023) *** Hay 635 acuerdos abiertos que han finalizado ... [Tareas.notifica_acuerdos_abiertos] Enviando notificación 2688 de acuerdo expirado (3257) </pre> Hay un error en la implementación (seguramente de un copia y pega), teniéndose que eliminar la línea: <pre> def acuerdo_expirado_abierto(usuario, acuerdo) ... @usuario_identificado = usuario_identificado </pre> Parece que la cantidad de correos a enviar es enorme. Acciones preventivas: 1.- Sólo enviar correos a los usuarios directamentre relacionados con los acuerdos. 2.- Se podría añadir una variable de configuración para enviar los correos o no.