fix: rethink logic for scheduled posts

This makes things cleaner and shows the right “time since” for scheduled
posts.

Author: m0wer

api/resolvers/item.js

@@ -260,8 +260,8 @@ export const muteClause = me =>
 
 export const scheduledOrMine = (me) => {
   return me
-    ? `("Item"."scheduledAt" IS NULL OR "Item"."userId" = ${me.id})`
-    : '"Item"."scheduledAt" IS NULL'
+    ? `("Item"."scheduledAt" IS NULL OR "Item"."scheduledAt" <= now() OR "Item"."userId" = ${me.id})`
+    : '("Item"."scheduledAt" IS NULL OR "Item"."scheduledAt" <= now())'
 }
 
 const HIDE_NSFW_CLAUSE = '("Sub"."nsfw" = FALSE OR "Sub"."nsfw" IS NULL)'
@@ -1125,12 +1125,10 @@ export default {
 
       const publishTime = new Date()
 
-      // Publish immediately with current timestamp
+      // Publish immediately - keep original createdAt and scheduledAt
       const updatedItem = await models.item.update({
         where: { id: Number(id) },
         data: {
-          scheduledAt: null,
-          createdAt: publishTime,
           updatedAt: publishTime
         }
       })

lib/apollo.js

@@ -332,6 +332,12 @@ function getClient (uri) {
 
                 return reactiveVar()
               }
+            },
+            // Ensure scheduled post state changes are properly reflected in cache
+            scheduledAt: {
+              merge (existing, incoming) {
+                return incoming
+              }
             }
           }
         }

worker/publishScheduledPosts.js

@@ -19,12 +19,10 @@ export async function publishScheduledPost ({ data: { itemId }, models, lnd }) {
 
   const publishTime = new Date()
 
-  // Update the item to be published with new timestamp
+  // Update the item to be published - keep original createdAt and scheduledAt
   await models.item.update({
     where: { id: itemId },
     data: {
-      scheduledAt: null,
-      createdAt: publishTime,
       updatedAt: publishTime
     }
   })