Skip to content

Add isSensitive prop to reader revenue components#15582

Open
juabara wants to merge 4 commits intomainfrom
jm/feat-is-sensitive
Open

Add isSensitive prop to reader revenue components#15582
juabara wants to merge 4 commits intomainfrom
jm/feat-is-sensitive

Conversation

@juabara
Copy link
Contributor

@juabara juabara commented Mar 24, 2026

This PR threads the isSensitive article property through the reader revenue component stack to enable sensitive content detection for epic and banner eligibility checks.

Changes:

  • Add isSensitive prop to LiveBlogEpic, SlotBodyEnd, and StickyBottomBanner components
  • Update canShowReaderRevenueEpic to receive and handle isSensitive in its decision data
  • Fix CrosswordLayout to use StickyBottomBanner (matching other layouts) instead of SlotBodyEnd
  • Fix TypeScript compilation errors in Island.test.tsx for missing required props

…ayouts.

- Standard, Showcase, Interactive, Immersive, Comment, Audio and LiveBlog.
- Update tests to include the new required prop.
@juabara juabara added the feature Departmental tracking: work on a new feature label Mar 24, 2026
@github-actions
Copy link

github-actions bot commented Mar 24, 2026

@juabara juabara marked this pull request as ready for review March 24, 2026 16:22
@juabara juabara requested a review from a team as a code owner March 24, 2026 16:22
@github-actions
Copy link

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@github-actions
Copy link

github-actions bot commented Mar 24, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant