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

Check Posts page setting when fetching archive link for news #204

Open
ashleykolodziej opened this issue Feb 14, 2020 · 1 comment
Open
Labels
🔥 Product bug A bug which is not directly related to a custom child theme.

Comments

@ashleykolodziej
Copy link
Contributor

Responsive Version:

2.0

Expected behavior:

News archive links should use the Posts page setting in Settings > Reading, if it is defined, for the news archive link.

Actual behavior:

Responsive checks for pages using a news template, and returns the URL of the first one that happens to be using a news page template in alphabetical order. This is problematic because in the case that a website uses the News page template more than once, there is no way to control which news page gets the View All designation. The news template is often used to show news from a specific category in multiple sections - so counting on there being only one is fine as a backup, but should not be the first stop for THIS IS THE NEWS PAGE TO RULE THEM ALLLLLLLL.

Link(s) where this can be reproduced:

https://www-staging.bu.edu/history/

Steps to reproduce:

  1. Set a page beginning with "A" to use the news page template.
  2. Set a page called News to use the news page template.
  3. Any "view all" link for News will return the URL for A, not News.
@ashleykolodziej ashleykolodziej added the 🔥 Product bug A bug which is not directly related to a custom child theme. label Feb 14, 2020
@crystalms
Copy link

This also appears to be impacting category links on single posts, if said category (for instance, Research) has a page template set to News and has the Research taxonomy selected in the News settings, it will replace the default wordpress category archive for Research with the page using the News template.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔥 Product bug A bug which is not directly related to a custom child theme.
Projects
None yet
Development

No branches or pull requests

2 participants