fix: notify_user need to accept user not uid (pinterest#1095)

* fix: notify_user need to accept user not uid * remove markdown link
lilyli9 · Dec 6, 2022 · 8b5b78f · 8b5b78f
1 parent 676320f
commit 8b5b78f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/querybook/notification_templates/schedule_disabled_notification.md b/querybook/notification_templates/schedule_disabled_notification.md
@@ -1,4 +1,4 @@
-Your scheduled DataDoc [{{ data_doc_title }}]({{ doc_url }}) has been disabled due to the following reason:
+Your scheduled DataDoc {{ data_doc_title }} {{ doc_url }} has been disabled due to the following reason:
 
 **{{ disabled_reason }}**
 

diff --git a/querybook/server/tasks/disable_scheduled_docs.py b/querybook/server/tasks/disable_scheduled_docs.py
@@ -165,7 +165,7 @@ def notify_schedule_owners(notifier: str, tasks_to_disable: list, session):
         for possible_owner in possible_owners:
             user = get_user_by_id(possible_owner, session=session)
             if not user.deleted:
-                doc_owner = possible_owner
+                doc_owner = user
                 break
 
         if doc_owner is None: