Skip to content

Conversation

obenland
Copy link
Member

@obenland obenland commented Oct 1, 2025

Proposed changes:

  • Corrects the URL path from '/reader/feed/' to '/reader/feeds/' when generating the WordPress.com reader feed link. This ensures the link points to the correct location.

Other information:

  • Have you written new tests for your changes, if applicable?

Testing instructions:

Corrects the URL path from '/reader/feed/' to '/reader/feeds/' when generating the WordPress.com reader feed link. This ensures the link points to the correct location.
@obenland obenland requested a review from pfefferle October 1, 2025 17:08
@obenland obenland self-assigned this Oct 1, 2025
@Copilot Copilot AI review requested due to automatic review settings October 1, 2025 17:08
@obenland obenland added the Skip Changelog Disables the "Changelog Updated" action for PRs where changelog entries are not necessary. label Oct 1, 2025
Copy link

@Copilot 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

This PR fixes an incorrect URL path in the Jetpack reader feed functionality by correcting the WordPress.com reader feed link from '/reader/feed/' to '/reader/feeds/'.

  • Updates the URL path to match WordPress.com's actual reader feed endpoint structure

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added the [Focus] Compatibility Ensuring the plugin plays well with other plugins label Oct 1, 2025
@pfefferle
Copy link
Member

Oh, do we need to hotfix that on DotCom?

Changed expected Reader feed URLs from '/reader/feed/' to '/reader/feeds/' in integration tests to match updated URL structure. Also updated logic for WPCOM environment setup to reflect the new URL pattern.
Copy link
Member

@pfefferle pfefferle left a comment

Choose a reason for hiding this comment

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

I fixed the unittests

@obenland obenland merged commit 5b7dd9a into trunk Oct 1, 2025
13 checks passed
@obenland obenland deleted the fix/typo branch October 1, 2025 18:21
pfefferle added a commit that referenced this pull request Oct 6, 2025
Co-authored-by: Matthias Pfefferle <pfefferle@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Focus] Compatibility Ensuring the plugin plays well with other plugins Skip Changelog Disables the "Changelog Updated" action for PRs where changelog entries are not necessary.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants