Skip to content

Commit fe683a6

Browse files
pnvasanthclaude
andcommitted
fix(feed): use versionToTimeFeedGenerator for authenticated TIME feed
The authenticated TIME feed should use exactly the same config as POPULARITY feed (versionToFeedGenerator), with only order_by: date added. This ensures consistent behavior between anonymous and authenticated feeds, using the proper Lofn config generator. Co-Authored-By: Claude Opus 4.5 <[email protected]>
1 parent 8feb9a8 commit fe683a6

File tree

3 files changed

+17
-1
lines changed

3 files changed

+17
-1
lines changed

src/integrations/feed/generators.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,3 +171,17 @@ export const versionToFeedGenerator = (version: number): FeedGenerator => {
171171
}),
172172
);
173173
};
174+
175+
export const versionToTimeFeedGenerator = (version: number): FeedGenerator => {
176+
return new FeedGenerator(
177+
feedClient,
178+
new FeedLofnConfigGenerator(
179+
{ ...baseFeedConfig, order_by: FeedOrderBy.Date },
180+
lofnClient,
181+
{
182+
...opts,
183+
feed_version: version.toString() as FeedVersion,
184+
},
185+
),
186+
);
187+
};

src/integrations/feed/index.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@ export {
88
FeedGenerator,
99
feedGenerators,
1010
versionToFeedGenerator,
11+
versionToTimeFeedGenerator,
1112
feedClient,
1213
} from './generators';

src/schema/feeds.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ import {
6161
FeedResponse,
6262
SimpleFeedConfigGenerator,
6363
versionToFeedGenerator,
64+
versionToTimeFeedGenerator,
6465
} from '../integrations/feed';
6566
import {
6667
AuthenticationError,
@@ -1461,7 +1462,7 @@ export const resolvers: IResolvers<unknown, BaseContext> = traceResolvers<
14611462
source,
14621463
{
14631464
...(args as FeedArgs),
1464-
generator: feedGenerators['time']!,
1465+
generator: versionToTimeFeedGenerator(args.version),
14651466
},
14661467
ctx,
14671468
info,

0 commit comments

Comments
 (0)