UX: Warn in /logs when PM is not sent (#271)

discourse · Dec 11, 2023 · 9bd7019 · 9bd7019
1 parent 3dd5ad0
commit 9bd7019
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/plugin.rb b/plugin.rb
@@ -116,7 +116,13 @@ module ::DiscourseDataExplorer
             DiscourseDataExplorer::ReportGenerator.new(automation.last_updated_by_id)
           report_pms = data_explorer_report.generate(query_id, query_params, recipients)
 
-          report_pms.each { |pm| utils.send_pm(pm, automation_id: automation.id) }
+          report_pms.each do |pm|
+            begin
+              utils.send_pm(pm, automation_id: automation.id)
+            rescue ActiveRecord::RecordNotSaved => e
+              Rails.logger.warn "#{DiscourseDataExplorer::PLUGIN_NAME} - couldn't send PM for automation #{automation.id}: #{e.message}"
+            end
+          end
         end
       end
     end