Buzzsprout episode URL update

Created on 21 October 2024, 8 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. β†’ (Merged) created by ekes
  • Pipeline finished with Failed
    8 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
    8 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
    8 months ago
    Total: 168s
    #316116
  • Status changed to Needs review 13 days ago
  • πŸ‡ΊπŸ‡ΈUnited States dgroene

    Patch for version 1.9.1 (same as MR)

  • πŸ‡ΊπŸ‡ΈUnited States dgroene
  • Pipeline finished with Skipped
    12 days ago
    #504504
  • Pipeline finished with Skipped
    12 days ago
    #504506
  • πŸ‡ͺπŸ‡ΈSpain marcoscano Barcelona, Spain

    Sorry for the late reply. Committed, thanks! πŸ‘

Production build 0.71.5 2024