From 0c0a95a79c7494eba9e0d16c429face43f4b04a1 Mon Sep 17 00:00:00 2001
From: Hongbo Wu <hongbo@omnivore.app>
Date: Wed, 24 Apr 2024 21:10:13 +0800
Subject: [PATCH] fix newsletter dir not saved correctly

---
 packages/api/src/services/save_email.ts | 4 +++-
 packages/content-fetch/src/job.ts       | 3 ---
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/packages/api/src/services/save_email.ts b/packages/api/src/services/save_email.ts
index 821dfdca4..699db6919 100644
--- a/packages/api/src/services/save_email.ts
+++ b/packages/api/src/services/save_email.ts
@@ -114,7 +114,9 @@ export const saveEmail = async (
       labelNames: labels.map((label) => label.name),
       itemLanguage: parseResult.parsedContent?.language,
       directionality:
-        (parseResult.parsedContent?.dir as DirectionalityType) || undefined,
+        parseResult.parsedContent?.dir?.toLowerCase() === 'rtl'
+          ? DirectionalityType.RTL
+          : DirectionalityType.LTR, // default to LTR
     },
     input.userId
   )
diff --git a/packages/content-fetch/src/job.ts b/packages/content-fetch/src/job.ts
index de3c2fb58..0a3a884ee 100644
--- a/packages/content-fetch/src/job.ts
+++ b/packages/content-fetch/src/job.ts
@@ -59,9 +59,6 @@ const getAttempts = (job: SavePageJob): number => {
 
 const getOpts = (job: SavePageJob): BulkJobOptions => {
   return {
-    jobId: `${JOB_NAME}_${job.userId}_${job.data.finalUrl}`, // make sure we don't have duplicate jobs
-    removeOnComplete: true,
-    removeOnFail: true,
     attempts: getAttempts(job),
     priority: getPriority(job),
     backoff: {