From 0444eeb3477b85523c808ae2ebd3e58801c61e1e Mon Sep 17 00:00:00 2001 From: Amar Trebinjac Date: Fri, 21 Nov 2025 16:52:36 +0100 Subject: [PATCH 1/2] refactor: no longer limit anonymous users on experience --- src/schema/profile.ts | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/schema/profile.ts b/src/schema/profile.ts index ee5c3628f..96967f9f6 100644 --- a/src/schema/profile.ts +++ b/src/schema/profile.ts @@ -236,11 +236,8 @@ export const resolvers = traceResolvers({ ctx, info, (nodeSize) => - !!ctx.userId && userExperiencesPageGenerator.hasPreviousPage(page, nodeSize), - (nodeSize) => - !!ctx.userId && - userExperiencesPageGenerator.hasNextPage(page, nodeSize), + (nodeSize) => userExperiencesPageGenerator.hasNextPage(page, nodeSize), (node, index) => userExperiencesPageGenerator.nodeToCursor(page, args, node, index), (builder) => { @@ -259,8 +256,8 @@ export const resolvers = traceResolvers({ builder.queryBuilder .orderBy(`${builder.alias}."endedAt"`, 'DESC', 'NULLS FIRST') .addOrderBy(`${builder.alias}."startedAt"`, 'DESC') - .limit(!ctx.userId ? 1 : getLimit({ limit: page.limit })) - .offset(!ctx.userId ? 0 : page.offset); + .limit(getLimit({ limit: page.limit })) + .offset(page.offset); return builder; }, From ce77059795f8795569611c57c7b720df9c23762d Mon Sep 17 00:00:00 2001 From: Amar Trebinjac Date: Fri, 21 Nov 2025 17:01:40 +0100 Subject: [PATCH 2/2] remove irrelevant test --- __tests__/schema/profile.ts | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/__tests__/schema/profile.ts b/__tests__/schema/profile.ts index 929373dfb..152d7ca84 100644 --- a/__tests__/schema/profile.ts +++ b/__tests__/schema/profile.ts @@ -185,17 +185,6 @@ describe('query userExperiences', () => { expect(res.data.userExperiences.pageInfo.hasNextPage).toBe(false); }); - it('should return only 1 experience for non-logged-in user', async () => { - loggedUser = null; - - const res = await client.query(USER_EXPERIENCES_QUERY, { - variables: { userId: '1' }, - }); - - expect(res.errors).toBeFalsy(); - expect(res.data.userExperiences.edges).toHaveLength(1); - }); - it('should return all fields for logged-in users', async () => { loggedUser = '1';