Skip to content

WS 2423 - Remove podcastEpisodeLinkedData toggle#13883

Merged
Louis-Matsika merged 4 commits intolatestfrom
WS-2423--podcast-SEO-toggles
Apr 9, 2026
Merged

WS 2423 - Remove podcastEpisodeLinkedData toggle#13883
Louis-Matsika merged 4 commits intolatestfrom
WS-2423--podcast-SEO-toggles

Conversation

@Louis-Matsika
Copy link
Copy Markdown
Contributor

@Louis-Matsika Louis-Matsika commented Apr 8, 2026

Resolves JIRA: WS-2423

Summary

Removes the podcastEpisodeLinkedData toggle

Changes

File Description
ws-nextjs-app/pages/[service]/onDemandAudio/OnDemandAudioLayout.tsx Removes toggle-gated logic; always enables PodcastEpisode schema for podcast episode pages.
ws-nextjs-app/pages/[service]/onDemandAudio/index.test.tsx Removes toggle-specific test setup/branch; keeps snapshot coverage for PodcastEpisode schema.
ws-nextjs-app/pages/[service]/onDemandAudio/snapshots/index.test.tsx.snap Removes snapshot corresponding to the toggle-off scenario.
src/app/lib/config/toggles/testConfig.js Removes podcastEpisodeLinkedData from test toggle config.
src/app/lib/config/toggles/localConfig.js Removes podcastEpisodeLinkedData from local toggle config.
src/app/lib/config/toggles/liveConfig.js Removes podcastEpisodeLinkedData from live toggle config.
src/app/lib/config/toggles/README.md Removes podcastEpisodeLinkedData from documented toggle list.
src/app/lib/config/toggles/snapshots/index.test.js.snap Removes podcastEpisodeLinkedData from toggle config snapshots.
.yarn/cache Yarn cache artefacts updated

Testing

To check that linked data still comes through visit a podcast episode page e.g. http://localhost:7081/russian/podcasts/p076qqzl/p0nbvf7d?renderer_env=live

in the browser console, search for the script application/ld+json

the field mainEntity @type should be present and have a value of PodcastEpisode

Screenshot 2026-04-08 at 11 18 06 am

Useful Links

@Louis-Matsika Louis-Matsika marked this pull request as ready for review April 8, 2026 10:30
Copilot AI review requested due to automatic review settings April 8, 2026 10:30
@Louis-Matsika Louis-Matsika requested a review from a team as a code owner April 8, 2026 10:30
@Louis-Matsika Louis-Matsika self-assigned this Apr 8, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the podcastEpisodeLinkedData feature toggle so podcast episode pages always emit PodcastEpisode linked data, and cleans up associated toggle config/docs/tests.

Changes:

  • Remove podcastEpisodeLinkedData toggle usage in the on-demand audio layout so PodcastEpisode schema emission is unconditional for podcast episode pages.
  • Remove the toggle from toggle configs and the toggles README.
  • Update on-demand audio tests/snapshots to remove toggle-on/off branching for linked data.

Reviewed changes

Copilot reviewed 7 out of 19 changed files in this pull request and generated no comments.

Show a summary per file
File Description
ws-nextjs-app/pages/[service]/onDemandAudio/OnDemandAudioLayout.tsx Removes toggle-gated logic; always enables PodcastEpisode schema for podcast episode pages.
ws-nextjs-app/pages/[service]/onDemandAudio/index.test.tsx Removes toggle-specific test setup/branch; keeps snapshot coverage for PodcastEpisode schema.
ws-nextjs-app/pages/[service]/onDemandAudio/snapshots/index.test.tsx.snap Removes snapshot corresponding to the toggle-off scenario.
src/app/lib/config/toggles/testConfig.js Removes podcastEpisodeLinkedData from test toggle config.
src/app/lib/config/toggles/localConfig.js Removes podcastEpisodeLinkedData from local toggle config.
src/app/lib/config/toggles/liveConfig.js Removes podcastEpisodeLinkedData from live toggle config.
src/app/lib/config/toggles/README.md Removes podcastEpisodeLinkedData from documented toggle list.
src/app/lib/config/toggles/snapshots/index.test.js.snap Removes podcastEpisodeLinkedData from toggle config snapshots.
.yarn/cache/fsevents-patch-6b67494872-10.zip Yarn cache artefact added (binary).
.yarn/cache/@unrs-resolver-binding-linux-x64-gnu-npm-1.11.0-f4a171e974-10.zip Yarn cache artefact updated/added (binary).
.yarn/cache/@img-sharp-linux-x64-npm-0.34.5-06b002e5f6-10.zip Yarn cache artefact updated/added (binary).
.yarn/cache/@img-sharp-libvips-linux-x64-npm-1.2.4-105fd6d44d-10.zip Yarn cache artefact updated/added (binary).
.yarn/cache/@esbuild-linux-x64-npm-0.27.2-11f1a3d9db-10.zip Yarn cache artefact updated/added (binary).

@Louis-Matsika Louis-Matsika changed the title remove podcastEpisodeLinkedData toggle WS 2423 - Remove podcastEpisodeLinkedData toggle Apr 8, 2026
@Louis-Matsika Louis-Matsika merged commit b8187db into latest Apr 9, 2026
13 checks passed
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.

4 participants