Skip to content

feat: show community take on the legacy post layout - #6507

Merged
rebelchris merged 1 commit into
mainfrom
feat/community-take-legacy-post-layout
Aug 21, 2026
Merged

feat: show community take on the legacy post layout#6507
rebelchris merged 1 commit into
mainfrom
feat/community-take-legacy-post-layout

Conversation

@rebelchris

@rebelchris rebelchris commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Problem

People reported not seeing community takes on posts that definitely have one.

The take is only rendered by PostFocusCard, which is mounted only when usePostRedesign().showRedesign is true. In GrowthBook, post_redesign has prod default false with a single force rule targeting {"team": 1} — so the surface is effectively team-only. The community_sentiment flag itself is fully on in prod and the API returns Post.communitySentiment ungated, so the data was there and just never rendered for anyone on the legacy layout.

Change

Render CommunitySentiment in the legacy PostContent layout as well:

  • full post page only (isPostPage), matching the redesign which hides it in the preview modal via !onClose
  • same useConditionalFeature enrollment on community_sentiment, with shouldEvaluate bound to the post actually having a take so take-less posts don't dilute the treatment/control split
  • isDevelopment escape hatch kept for local preview; the committed flag default stays false

Frontend-only — generation is unconditional, no backfill needed.

Notes

  • Modal behaviour is unchanged in both layouts. If we want takes in the feed preview modal too, that's a separate call (drop !onClose / isPostPage).
  • Node/pnpm aren't available in this runtime, so lint/typecheck/tests weren't run locally — relying on CI.

🤖 Opened by Smith (AI) on behalf of Chris.

Preview domain

https://feat-community-take-legacy-post.preview.app.daily.dev

The community take only rendered inside PostFocusCard, which is gated by
post_redesign — currently forced on for team accounts only. That made the
surface invisible to everyone else even though community_sentiment is fully
rolled out and the backend generates takes unconditionally.

Render it in the legacy PostContent layout too, on the full post page only
(matching the redesign, which hides it in the preview modal), with the same
conditional enrollment so take-less posts don't dilute the split.
@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Aug 21, 2026 6:33pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Aug 21, 2026 6:33pm

Request Review

@rebelchris
rebelchris merged commit 412d322 into main Aug 21, 2026
11 checks passed
@rebelchris
rebelchris deleted the feat/community-take-legacy-post-layout branch August 21, 2026 18:36
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