Adding sentryHEADmaster

1 files changed, 6 insertions, 4 deletions

diff --git a/maconomy_hours.py b/maconomy_hours.py
index 608d502..4b20c7f 100644
--- a/maconomy_hours.py
+++ b/maconomy_hours.py
@@ -1,7 +1,6 @@
 from maconomy import cli, create_db, TimeRegistrationRepository, Mailer
 from maconomy.views import EmployeeEmailView, ManagerEmailView, CEOEmailView
 from maconomy import utils
-from collections import defaultdict
 
 
 def main(config, options):
@@ -18,6 +17,9 @@ def main(config, options):
         summary(timesheets)
     else:
         normal(timesheets, config, mailer)
+        missing_mail = [ts.employee for ts in timesheets if utils.is_empty(ts.employee.email)]
+        if missing_mail:
+            utils.sentry_log("Employees missing emails", {'emplyees': [str(e) for e in missing_mail]})
 
     # Close stuff
     if mailer:
@@ -30,18 +32,17 @@ def normal(timesheets, config, mailer):
     view = EmployeeEmailView(config)
     for timesheet in timesheets:
         if not timesheet.is_submitted():
-            mail = view.render(timesheet) 
+            mail = view.render(timesheet)
             subject = u"Timesheet reminder for {}".format(timesheet.employee)
             to = timesheet.employee.email
             if mailer:
                 if to and to.strip():
                     mailer.send(to, subject, mail)
-                else:
-                    print(u"No email for: {}".format(timesheet.employee))
             else:
                 print(subject)
                 print(mail)
 
+
 def manager(timesheets, config, mailer):
     view = ManagerEmailView(config)
     # extract employees for manager email lookup
@@ -84,4 +85,5 @@ def summary(timesheets):
 if __name__ == '__main__':
     args = cli.parse()
     config = cli.load_config(args.config)
+    utils.sentry(config)
     main(config, args)