From 8ac27171c75e26266ad5575ce5551fb471d44eaa Mon Sep 17 00:00:00 2001 From: Hongbo Wu Date: Wed, 10 Jul 2024 10:51:50 +0800 Subject: [PATCH 1/2] fix: include just added items in top picks --- packages/api/src/jobs/update_home.ts | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/packages/api/src/jobs/update_home.ts b/packages/api/src/jobs/update_home.ts index 9a0f1e94a..40e79ae21 100644 --- a/packages/api/src/jobs/update_home.ts +++ b/packages/api/src/jobs/update_home.ts @@ -146,7 +146,6 @@ const getJustAddedCandidates = async ( const selectCandidates = async ( user: User, - excludes: Array = [], limit = 100 ): Promise> => { const userId = user.id @@ -155,7 +154,7 @@ const selectCandidates = async ( { size: limit, includeContent: false, - query: `in:inbox -is:seen -includes:${excludes.join(',')}`, + query: 'in:inbox -is:seen', }, userId ) @@ -517,10 +516,7 @@ export const updateHome = async (data: UpdateHomeJobData) => { logger.info(`Found ${justAddedCandidates.length} just added candidates`) end = latency.startTimer({ step: 'select' }) - const candidates = await selectCandidates( - user, - justAddedCandidates.map((c) => c.id) - ) + const candidates = await selectCandidates(user) end() logger.info(`Found ${candidates.length} candidates`) From 1aaa4202fcd66a8cb29b18c33696f82e0ed27487 Mon Sep 17 00:00:00 2001 From: Hongbo Wu Date: Wed, 10 Jul 2024 10:54:36 +0800 Subject: [PATCH 2/2] move items which have less than 250 words to quick links section --- packages/api/src/jobs/update_home.ts | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/packages/api/src/jobs/update_home.ts b/packages/api/src/jobs/update_home.ts index 40e79ae21..f59d1e3d9 100644 --- a/packages/api/src/jobs/update_home.ts +++ b/packages/api/src/jobs/update_home.ts @@ -428,8 +428,7 @@ const mixHomeItems = ( const topCandidates = rankedHomeItems.slice(0, 50) - // find the median word count - const wordCountThreshold = 500 + const wordCountThreshold = 250 // separate items into two groups based on word count const shortItems: Array = [] const longItems: Array = []