fix(nx-dev): pinning logic on blog entry page

juristr · FrozenPandaz · commit c747d3f172e1 · 2025-07-02T14:38:08.000-04:00
(cherry picked from commit ebb1716)
diff --git a/nx-dev/data-access-documents/src/lib/blog.util.ts b/nx-dev/data-access-documents/src/lib/blog.util.ts
@@ -2,8 +2,8 @@ import { BlogPostDataEntry } from './blog.model';
 
 export function sortPosts(posts: BlogPostDataEntry[]): BlogPostDataEntry[] {
   return posts.sort((a, b) => {
-    if (a.pinned && !b.pinned) return -1;
-    if (b.pinned && !a.pinned) return 1;
+    if (a.pinned === true && b.pinned !== true) return -1;
+    if (b.pinned === true && a.pinned !== true) return 1;
     return new Date(b.date).getTime() - new Date(a.date).getTime();
   });
 }
diff --git a/nx-dev/ui-blog/src/lib/blog-container.tsx b/nx-dev/ui-blog/src/lib/blog-container.tsx
@@ -30,14 +30,17 @@ export interface BlogContainerProps {
 export function sortFirstFivePosts(
   posts: BlogPostDataEntry[]
 ): BlogPostDataEntry[] {
-  // Separate posts: pinned-able posts first
-  const allowedPinnedPosts = posts.filter((p) => p.pinned !== false);
+  // Sort posts: pinned posts first, then by date
+  const sortedPosts = posts.sort((a, b) => {
+    // If one is pinned and the other isn't, prioritize the pinned one
+    if (a.pinned === true && b.pinned !== true) return -1;
+    if (b.pinned === true && a.pinned !== true) return 1;
 
-  allowedPinnedPosts.sort(
-    (a, b) => new Date(b.date).valueOf() - new Date(a.date).valueOf()
-  );
+    // Otherwise, sort by date (newest first)
+    return new Date(b.date).valueOf() - new Date(a.date).valueOf();
+  });
 
-  return allowedPinnedPosts.slice(0, 5);
+  return sortedPosts.slice(0, 5);
 }
 
 export function BlogContainer({ blogPosts, tags }: BlogContainerProps) {
