Just sort, no limit, on collection page

cloudflare · Jan 8, 2025 · eef7a29 · eef7a29
1 parent c54bcf7
commit eef7a29
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/app/routes/built-with.collections.$slug.jsx b/app/routes/built-with.collections.$slug.jsx
@@ -5,7 +5,7 @@ import Layout from "../components/layout";
 import Project from "../components/project";
 import { isInitialRoute } from "../components/route-update-history.js";
 
-import { sortAndLimitProjects } from "../utils";
+import { sortProjects } from "../utils";
 
 import { client } from "../lib/sanity";
 
@@ -59,7 +59,7 @@ const BuiltWithPage = () => {
         <div className="Collections">
           <div className="Collection">
             <div className="Collection--projects">
-              {sortAndLimitProjects(collection)
+              {sortProjects(collection)
                 .map((project, pi) => (
                   <div className="Collection--project" key={pi}>
                     <Project

diff --git a/app/utils.jsx b/app/utils.jsx
@@ -126,6 +126,10 @@ const resultTemplate = () => `
 
 const sortAndLimitProjects = (collection) => {
   const numToShow = collection.numProjectsToShow || 3;
+  return sortProjects(collection).slice(0, numToShow)
+}
+
+const sortProjects = (collection) => {
   const sortOrder = collection.sortOrder || "_updated_at desc";
   const [sortField, sortDirection] = sortOrder.split(" ");
 
@@ -140,18 +144,16 @@ const sortAndLimitProjects = (collection) => {
       // Not sure, default to created_at desc
       return a[sortField] > b[sortField] ? 1 : -1;
     })
-    .slice(0, numToShow)
 
   return projects;
 }
 
-
-
 export {
   PROJECTS_PER_COLLECTION,
   flatten,
   normalizeCollection,
   resultTemplate,
+  sortProjects,
   sortAndLimitProjects,
   useLocalStorage,
   useSSR,