Merge pull request #349 from unicolet/issue_348_notify_for_remind

#348 Scheduled reminders should notify the user who requested them
skoczen · Jun 13, 2018 · 81b9ef8 · 81b9ef8
2 parents cf7d53b + 18048c0
commit 81b9ef8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/AUTHORS b/AUTHORS
@@ -52,3 +52,5 @@ TaunoTinits, https://github.com/TaunoTinits
 ostracon, https://github.com/ostracon
 mattcl, https://github.com/mattcl
 Boris Peterbarg, https://github.com/reist
+unicolet, https://github.com/unicolet
+
diff --git a/will/plugins/productivity/remind.py b/will/plugins/productivity/remind.py
@@ -19,7 +19,7 @@ def remind_me_at(self, message, reminder_text=None, remind_time=None, to_string=
             "reminder_text": reminder_text,
             "to_string": formatted_to_string,
         }
-        self.schedule_say(formatted_reminder_text, parsed_time, message=message)
+        self.schedule_say(formatted_reminder_text, parsed_time, message=message, notify=True)
         self.say("%(reminder_text)s %(natural_datetime)s. Got it." % locals(), message=message)
 
     @respond_to("(?:can|will you )?remind (?P<reminder_recipient>(?!me).*?)(?P<to_string> to>) ?(?P<reminder_text>.*?) (at|on|in) (?P<remind_time>.*)?\??")
@@ -39,5 +39,5 @@ def remind_somebody_at(self, message, reminder_recipient=None, reminder_text=Non
                 "to_string": formatted_to_string,
             }
 
-        self.schedule_say(formatted_reminder_text, parsed_time, message=message)
+        self.schedule_say(formatted_reminder_text, parsed_time, message=message, notify=True)
         self.say("%(reminder_text)s %(natural_datetime)s. Got it." % locals(), message=message)