search

PRX / feeder.prx.org

Dovetail podcast content management system
https://podcasts.dovetail.prx.org
GNU Affero General Public License v3.0
5 stars 0 forks source link

ITunes Summary handling #949

Closed cavis closed 9 months ago

cavis commented 10 months ago

In the new Feeder UI, we decided not to include the itunes:summary field anymore. Since it no longer appears in the apple docs.

But it turns out - Apple Podcasts is still using/preferring that field, over the description. What to do? Things to decide:

  1. The PodcastRSSImporter still pulls in the summary - should we stop doing that?
  2. For shows transitioning from Publish to Feeder UI, they won't be able to edit the existing summary. Should we just nil the field out, when you save from the non-API controller? Or what?
  3. The RSS builder still falls back to sanitizing the description, to render the <itunes:summary> flag. Should we stop doing that?
cavis commented 9 months ago

Todo: look at popular podcast RSS feeds, and see if they have itunes:summary tags. If not, we can probably just drop the field.

kookster commented 9 months ago

The Daily on simplecast still has it:

andrew@Andrews-PRX-MacBook-Pro wfmt-services % curl -s https://feeds.simplecast.com/54nAGcIl  | grep "<itunes:summary" | wc -l
    2001
andrew@Andrews-PRX-MacBook-Pro wfmt-services % curl -s https://feeds.simplecast.com/54nAGcIl  | grep "<guid" | wc -l
    2000
brandonhundt commented 9 months ago

Darknet on Megaphone still has it too, but the content is the same, only with all <a> tags stripped

https://feeds.megaphone.fm/darknetdiaries

brandonhundt commented 9 months ago

Discussion here: https://prx.slack.com/archives/G0FKLK2Q5/p1705951262864649