Skip to content

feat(block-theme): move coauthors RSS function to the Newspack Plugin#2671

Open
laurelfulford wants to merge 1 commit intotrunkfrom
feat/move-coauthors-rss-function
Open

feat(block-theme): move coauthors RSS function to the Newspack Plugin#2671
laurelfulford wants to merge 1 commit intotrunkfrom
feat/move-coauthors-rss-function

Conversation

@laurelfulford
Copy link
Copy Markdown
Contributor

@laurelfulford laurelfulford commented Apr 4, 2026

All Submissions:

Changes proposed in this Pull Request:

This PR is a companion to Automattic/newspack-plugin#4629; they should be tested together.

Closes NPPD-1345

How to test the changes in this Pull Request:

  1. Follow the steps in feat(block-theme): move co-authors RSS feed code to the Newspack Plugin newspack-plugin#4629.

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

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

This PR removes the theme-level implementation that injected Co-Authors Plus author lists into RSS/feeds, with the intent that this responsibility now lives in the Newspack Plugin.

Changes:

  • Removed the newspack_coauthors_in_rss() filter callback.
  • Removed the add_filter( 'the_author', ... ) hook that altered feed author output.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@laurelfulford laurelfulford added the [Status] Needs Review The issue or pull request needs to be reviewed label Apr 4, 2026
@laurelfulford laurelfulford marked this pull request as ready for review April 4, 2026 01:17
@laurelfulford laurelfulford requested a review from a team as a code owner April 4, 2026 01:17
@laurelfulford laurelfulford changed the title feat: move coauthors RSS function to the Newspack Plugin feat(block-theme): move coauthors RSS function to the Newspack Plugin Apr 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Status] Needs Review The issue or pull request needs to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants