Skip to content

chore: remove post_page_feed feature flag (post page ends at the comments) - #7339

Merged
idoshamun merged 1 commit into
mainfrom
cleanup/post_page_feed
Nov 11, 2026
Merged

idoshamun merged 1 commit into
mainfrom
cleanup/post_page_feed

Conversation

@idoshamun

@idoshamun idoshamun commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

What

Removes the post_page_feed feature flag. The experiment was rolled back, so the post page keeps the control behaviour: false, the page ends at the comments, with no "More for you" feed below them and the post widgets visible at every width.

Result: the DoD retention goal was +2.0% (90%) [-1.1, +5.1]. Phase 2 lost phase 1's post-click lift: Any Post Click +0.4% [-1.8, +2.5]. Ad CTR and time on site leaned negative.

Deleted

  • featurePostPageFeed and the page's evaluation of it (pages/posts/[id]/index.tsx)
  • components/post/feed/PostPageFeed (component, CSS module, spec)
  • The belowComments slot, which was threaded through BasePostContent, PostContent, SquadPostContent, SocialTwitterPostContent, PollPostContent and CollectionPostContent, and getPostWidgetsClassName. Both only served the feed.
  • Feed's footer and errorScreen props and their specs. The post page feed was their only user.
  • The capitalize prop on SquadDirectoryNavbarItem (no remaining users), and the chip class exports from UnifiedMobileFeedNav are no longer exported.
  • The Storybook stories for the experiment (stories/experiments/PostPageFeed.stories.tsx, stories/experiments/postPageFeed/)
  • The post page feed cases in webapp/__tests__/PostPage.tsx

Kept: the useFeed empty-feed handling for excludedPostIds from #7018. MainFeedLayout still uses excludedPostIds.

Checks

  • oxlint and oxfmt --check pass on the touched files in shared and webapp
  • tsc passes in webapp. In shared, the only error is the existing unused @ts-expect-error in archive/ArchiveFeedPage.spec.tsx, which this PR doesn't touch.
  • jest: shared Feed.spec, components/post/**, squads layout and feeds nav: 40 suites, 323 tests pass. webapp __tests__/PostPage.tsx: 64 tests pass.
  • No references to post_page_feed, featurePostPageFeed, PostPageFeed, belowComments or getPostWidgetsClassName remain

Prepared by Smith on behalf of @idoshamun.

Preview domain

https://cleanup-post-page-feed.preview.app.daily.dev

@vercel

vercel Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
daily-webapp Ready Ready Preview Nov 11, 2026 6:14am UTC
storybook Ready Ready Preview Nov 11, 2026 6:14am UTC

Request Review

@idoshamun
idoshamun enabled auto-merge (squash) November 11, 2026 06:11
@idoshamun
idoshamun merged commit 2f9266e into main Nov 11, 2026
13 checks passed
@idoshamun
idoshamun deleted the cleanup/post_page_feed branch November 11, 2026 06:15

This branch was successfully deployed

2 active deployments
Preview – daily-webapp — ef1b291f Deployed Nov 11, 2026 by vercel[bot]
Preview – storybook — ef1b291f Deployed Nov 11, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant