Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Regular user feeds 404 #126

Closed
renatonascalves opened this issue Jul 10, 2023 · 3 comments
Closed

Regular user feeds 404 #126

renatonascalves opened this issue Jul 10, 2023 · 3 comments
Labels
wontfix This will not be worked on

Comments

@renatonascalves
Copy link
Contributor

Description of the bug

Currently, the plugin is removing the author rewrite rules, including support for the feed of regular authors in favor of the Byline profile feed.

add_filter( 'author_rewrite_rules', '__return_empty_array' );

This creates a regression where regular authors, which had feeds, lose their feed. And even if the user is linked to a profile, their legacy posts are not ported over to the new feed page (profile feed).

I'd recommend keeping the author feeds.

Steps To Reproduce

  1. Site already have authors with posts.
  2. Add Byline Manager
  3. The feed page for regular WP_User(s) 404s: http://site/author/user_slug/feed/

Additional Information

No response

@renatonascalves renatonascalves added the bug Something isn't working label Jul 10, 2023
@kevinfodness kevinfodness changed the title Regular user feeds 404 Byline Author Feeds Don't Include Posts By That Author Oct 16, 2024
@kevinfodness kevinfodness changed the title Byline Author Feeds Don't Include Posts By That Author Regular user feeds 404 Oct 16, 2024
@kevinfodness
Copy link
Member

This behavior is by design. Byline Manager replaces all aspects of authorship on the front-end, and Byline Manager author feeds should be used instead of normal WordPress user feeds. If you are adopting Byline Manager and want to have the URLs be the same, match the author slug from the user to the Byline Manager Profile. However, there is currently a bug #33 where the feeds for profiles do not contain posts authored by those profiles, which does need to be fixed, but I will close this issue in favor of that one.

@kevinfodness kevinfodness added wontfix This will not be worked on and removed bug Something isn't working labels Oct 16, 2024
@renatonascalves
Copy link
Contributor Author

If you are adopting Byline Manager and want to have the URLs be the same, match the author slug from the user to the Byline Manager Profile.

The use case at the time was that not all authors needed to have byline profiles. So those legacy authors posts/pages were gone. It does sort of force a site to choose where, for business reasons, they might not want to.

@kevinfodness
Copy link
Member

At a core plugin level, it's designed to (and should remain designed to) fully take over the author experience as far as users are concerned. If individual sites want to support a hybrid model, they can do that using custom code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants