Merge pull request #3860 from omnivore-app/fix/newsletter-dir

fix newsletter dir not saved correctly

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
   )

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: {