Merge pull request #930 from stackernews/921-pinned-posts-in-hot

Don't filter by pins in home
2024-03-17 12:51:45 -05:00 · 2024-03-17 12:51:45 -05:00 · 60cf7760d7
parent 1b2bb6d0fb 3e56bbe9c0
commit 60cf7760d7
1 changed files with 2 additions and 1 deletions
--- a/api/resolvers/item.js
+++ b/api/resolvers/item.js
@ -457,7 +457,8 @@ export default {
                    LEFT JOIN "Sub" ON "Sub"."name" = "Item"."subName"
                    ${joinZapRankPersonalView(me, models)}
                    ${whereClause(
-                      '"Item"."pinId" IS NULL',
+                      // in "home" (sub undefined), we want to show pinned items (but without the pin icon)
+                      sub ? '"Item"."pinId" IS NULL' : '',
                      '"Item"."deletedAt" IS NULL',
                      '"Item"."parentId" IS NULL',
                      '"Item".bio = false',