default sort by savedAt to avoid recent read page to appear on top of… (#254)

* default sort by savedAt to avoid recent read page to appear on top of the list

* revert lint on generated code

packages/api/src/elastic/index.ts

@@ -393,7 +393,8 @@ export const searchPages = async (
       inFilter,
     } = args
     const sortOrder = sort?.order === SortOrder.Ascending ? 'asc' : 'desc'
-    const sortField = sort?.by === SortBy.Score ? '_score' : 'updatedAt'
+    // default sort by saved_at
+    const sortField = sort?.by === SortBy.Score ? '_score' : 'savedAt'
     const includeLabels = labelFilters.filter(
       (filter) => filter.type === LabelFilterType.INCLUDE
     )

packages/api/src/generated/graphql.ts

@@ -1512,6 +1512,7 @@ export type SignupSuccess = {
 };
 
 export enum SortBy {
+  SavedAt = 'SAVED_AT',
   Score = 'SCORE',
   UpdatedTime = 'UPDATED_TIME'
 }

packages/api/src/generated/schema.graphql

@@ -1133,6 +1133,7 @@ type SignupSuccess {
 }
 
 enum SortBy {
+  SAVED_AT
   SCORE
   UPDATED_TIME
 }

packages/api/src/schema.ts

@@ -27,6 +27,7 @@ const schema = gql`
   enum SortBy {
     UPDATED_TIME
     SCORE
+    SAVED_AT
   }
 
   enum ContentReader {

packages/api/src/utils/search.ts

@@ -123,15 +123,16 @@ const parseSortParams = (str?: string): SortParams | undefined => {
     order?.toUpperCase() === 'ASC' ? SortOrder.Ascending : SortOrder.Descending
 
   switch (sort.toUpperCase()) {
-    case 'UPDATED_AT':
+    case 'UPDATED_TIME':
+    case 'SAVED_AT':
       return {
-        by: SortBy.UpdatedTime,
+        by: SortBy.SavedAt,
         order: sortOrder,
       }
     case 'SCORE':
+      // sort by score does not need an order
       return {
         by: SortBy.Score,
-        order: sortOrder,
       }
   }
 }