fix styling on suggested users, remove filter on which subscribed com…

…ments are shown
ForumMagnum · May 15, 2024 · dc49213 · dc49213
1 parent 824509c
commit dc49213
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/packages/lesswrong/components/subscriptions/SuggestedFeedSubscriptions.tsx b/packages/lesswrong/components/subscriptions/SuggestedFeedSubscriptions.tsx
@@ -17,7 +17,7 @@ const styles = (theme: ThemeType) => ({
     marginTop: 10,
     marginBottom: 10,
     background: theme.palette.panelBackground.recentDiscussionThread,
-    paddingTop: 18,
+    paddingTop: 12,
     paddingLeft: 16,
     paddingRight: 16,
     paddingBottom: 12,

diff --git a/packages/lesswrong/server/repos/PostsRepo.ts b/packages/lesswrong/server/repos/PostsRepo.ts
@@ -788,7 +788,7 @@ class PostsRepo extends AbstractRepo<"Posts"> {
   async getPostsAndCommentsFromSubscriptions(userId: string, numDays: number): Promise<Array<PostAndCommentsResultRow >> {
     return await this.getRawDb().manyOrNone<PostAndCommentsResultRow>(`
       WITH user_subscriptions AS (
-      SELECT DISTINCT "displayName", type, "documentId" AS "userId"
+      SELECT DISTINCT type, "documentId" AS "userId"
       FROM "Subscriptions" s
                LEFT JOIN "Users" u ON u._id = s."documentId"
       WHERE state = 'subscribed'
@@ -816,7 +816,8 @@ class PostsRepo extends AbstractRepo<"Posts"> {
           FROM "Comments" c
                JOIN user_subscriptions us USING ("userId")
           WHERE c."postedAt" > CURRENT_TIMESTAMP - INTERVAL $2
-          AND (c."baseScore" >= 5 OR (c.contents->>'wordCount')::numeric >= 200)
+          -- TODO: maybe reintroduce this filter?
+          -- AND (c."baseScore" >= 5 OR (c.contents->>'wordCount')::numeric >= 200)
           AND c.deleted IS NOT TRUE
           AND c."authorIsUnreviewed" IS NOT TRUE
           AND c.retracted IS NOT TRUE