Skip to content
Snippets Groups Projects
Unverified Commit 6eb6267c authored by Maz Ameli's avatar Maz Ameli Committed by GitHub
Browse files

Merge pull request #7353 from marcua/set-up-an-alert

Quick grammar/spelling fix for alert setup
parents 5cff17ec edf8dc3e
Branches
Tags
No related merge requests found
......@@ -319,7 +319,7 @@
(defn send-new-alert-email!
"Send out the initial 'new alert' email to the `CREATOR` of the alert"
[{:keys [creator] :as alert}]
(send-email! creator "You setup an alert" new-alert-template
(send-email! creator "You set up an alert" new-alert-template
(default-alert-context alert alert-condition-text)))
(defn send-you-unsubscribed-alert-email!
......
......@@ -110,7 +110,7 @@
~@body)))
(defn- rasta-new-alert-email [body-map]
(et/email-to :rasta {:subject "You setup an alert",
(et/email-to :rasta {:subject "You set up an alert",
:body (merge {"https://metabase.com/testmb" true,
"My question" true}
body-map)}))
......@@ -185,7 +185,7 @@
(assoc :creator (user-details :crowberto))
(assoc-in [:card :include_csv] true)
(update-in [:channels 0] merge {:schedule_hour 12, :schedule_type "daily", :recipients (set (map recipient-details [:rasta :crowberto]))}))
(merge (et/email-to :crowberto {:subject "You setup an alert",
(merge (et/email-to :crowberto {:subject "You set up an alert",
:body {"https://metabase.com/testmb" true,
"My question" true
"now getting alerts" false
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment