From 90b0845909a3a61584c190f262b93a831d850d87 Mon Sep 17 00:00:00 2001
From: Hongbo Wu <hongbo@omnivore.app>
Date: Thu, 9 May 2024 10:45:13 +0800
Subject: [PATCH 1/2] add "Update digest preference" button to the digest email

---
 packages/api/src/jobs/ai/create_digest.ts | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/packages/api/src/jobs/ai/create_digest.ts b/packages/api/src/jobs/ai/create_digest.ts
index f85ced5b6..0117e66f2 100644
--- a/packages/api/src/jobs/ai/create_digest.ts
+++ b/packages/api/src/jobs/ai/create_digest.ts
@@ -602,7 +602,6 @@ const sendEmail = async (user: User, digest: Digest) => {
 
       <button style="background-color: #FFEAA0;
                     border: 0px solid transparent;
-                    color: rgb(42, 42, 42);
                     padding:15px 32px;
                     font-size: 14px;
                     margin: 20px 0;
@@ -610,6 +609,20 @@ const sendEmail = async (user: User, digest: Digest) => {
                     border-radius: 5px;">
         <a href="${env.client.url}/digest/${digest.id}">Read in Omnivore</a>
       </button>
+
+      <button style="
+          font-size: 14px;
+          margin: 20px 0;
+          border: 0px solid transparent;
+          padding: 15px 32px;
+          font-size: 14px;
+          margin: 20px 0;
+          font-family: Inter, sans-serif;
+          border-radius: 5px;
+      "><a href="${
+        env.client.url
+      }/settings/account">Update digest preferences</a>
+      </button>
     </div>`
 
   await enqueueSendEmail({

From 5a730f12e7109dade3074f8063875a44d19f114e Mon Sep 17 00:00:00 2001
From: Hongbo Wu <hongbo@omnivore.app>
Date: Thu, 9 May 2024 10:53:57 +0800
Subject: [PATCH 2/2] only include the "Read in Omnivore" button if the digest
 is saved in library

---
 packages/api/src/jobs/ai/create_digest.ts | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/packages/api/src/jobs/ai/create_digest.ts b/packages/api/src/jobs/ai/create_digest.ts
index 0117e66f2..5f2ad7988 100644
--- a/packages/api/src/jobs/ai/create_digest.ts
+++ b/packages/api/src/jobs/ai/create_digest.ts
@@ -575,11 +575,12 @@ const sendPushNotification = async (userId: string, digest: Digest) => {
   await sendPushNotifications(userId, notification, 'reminder', data)
 }
 
-const sendEmail = async (user: User, digest: Digest) => {
+const sendEmail = async (user: User, digest: Digest, channels: Channel[]) => {
   const title = `${AUTHOR} ${new Date().toLocaleDateString()}`
   const subTitle = truncate(digest.title, { length: 200 }).slice(
     AUTHOR.length + 1
   )
+  const isInLibrary = channels.includes('library')
 
   const chapters = digest.chapters ?? []
 
@@ -600,15 +601,19 @@ const sendEmail = async (user: User, digest: Digest) => {
           )
           .join('')}
 
-      <button style="background-color: #FFEAA0;
+      ${
+        isInLibrary
+          ? `<button style="background-color: #FFEAA0;
                     border: 0px solid transparent;
                     padding:15px 32px;
                     font-size: 14px;
                     margin: 20px 0;
                     font-family: Inter, sans-serif;
                     border-radius: 5px;">
-        <a href="${env.client.url}/digest/${digest.id}">Read in Omnivore</a>
-      </button>
+              <a href="${env.client.url}/digest/${digest.id}">Read in Omnivore</a>
+            </button>`
+          : ''
+      }
 
       <button style="
           font-size: 14px;
@@ -619,9 +624,10 @@ const sendEmail = async (user: User, digest: Digest) => {
           margin: 20px 0;
           font-family: Inter, sans-serif;
           border-radius: 5px;
-      "><a href="${
-        env.client.url
-      }/settings/account">Update digest preferences</a>
+      ">
+        <a href="${
+          env.client.url
+        }/settings/account">Update digest preferences</a>
       </button>
     </div>`
 
@@ -712,7 +718,7 @@ const sendToChannels = async (
         case 'push':
           return sendPushNotification(user.id, digest)
         case 'email':
-          return sendEmail(user, digest)
+          return sendEmail(user, digest, deduplicateChannels)
         case 'library':
           return moveDigestToLibrary(user, digest)
         default: