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

Added another portion to largo_category_archive_posts conditional #1668

Merged

Conversation

joshdarby
Copy link
Collaborator

@joshdarby joshdarby commented Apr 19, 2019

Added another check to the largo_category_archive_posts function conditional to make sure we are not on a category feed for issue #1598.

Josh Darby added 2 commits April 19, 2019 10:38
Added another check to the `largo_category_archive_posts` function conditional to make sure we are not on a category feed
@benlk
Copy link
Collaborator

benlk commented Apr 19, 2019

Tested between 0.5-dev and this branch, following conclusions:

  • this branch shows the category top story in the feed
  • this branch shows the category featured posts in the feed
  • no changes were seen on the category archive page

@benlk benlk merged commit 85477c2 into 0.5-dev Apr 19, 2019
@benlk benlk deleted the 1598-featured-in-category-removes-item-from-category-rss-feed branch April 19, 2019 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants