Buzzsprout episode URL update

Created on 21 October 2024, 6 months ago

Problem/Motivation

The URL for episodes on Buzzsprout has been updated. As shown in their supported WP plugin:
https://plugins.trac.wordpress.org/changeset/3161694/

Steps to reproduce

Eg. go to the podcast that this person suggested 📌 Improve the "Configuration & Use" section Active on their rather Buzzsprout specific readme change. It was https://www.buzzsprout.com/1209764/10394989 when they wrote it, but it now redirects to https://www.buzzsprout.com/1209764/episodes/10394989

Proposed resolution

Make the minimum viable change to the regex to match the updated URL pattern.

Remaining tasks

Merge request to follow.

🐛 Bug report
Status

Active

Version

1.0

Component

Code

Created by

🇳🇱Netherlands ekes

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @ekes
  • Merge request !25Issue #3482122: New Buzzsprout episode URL. → (Open) created by ekes
  • Pipeline finished with Failed
    6 months ago
    Total: 217s
    #315832
  • 🇳🇱Netherlands ekes
  • 🇳🇱Netherlands ekes

    OK a bit more than the minimum change to the regex. Add the test and user feedback too.

  • Pipeline finished with Success
    6 months ago
    Total: 152s
    #315850
  • 🇪🇸Spain marcoscano Barcelona, Spain

    Thanks for filing this and for the fix.
    Is there a scenario where old URLs should be supported too? In other words, should be change the regex to the new format or should we just accept the new format as well?

  • 🇳🇱Netherlands ekes

    Thanks for filing this and for the fix.
    Is there a scenario where old URLs should be supported too? In other words, should be change the regex to the new format or should we just accept the new format as well?

    I think I was planning that (?:\/episodes) but then didn't follow through ?. As seen in the in the update from Buzzsprout themselves it's only the new format for their content. In our case however, there will be entities with the old pattern that have it in the field with the old URL, no point in updating that.

  • Pipeline finished with Success
    6 months ago
    Total: 168s
    #316116
Production build 0.71.5 2024