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.

Sign up for GitHub

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

Jump to bottom

Combining multiple collections into the main feed #390

Open
iBug opened this issue Apr 4, 2023 · 3 comments · May be fixed by jekyll/jekyll#9333
Open

Combining multiple collections into the main feed #390

iBug opened this issue Apr 4, 2023 · 3 comments · May be fixed by jekyll/jekyll#9333

Comments

@iBug
Copy link

iBug commented Apr 4, 2023

Basically a re-request from #289.

Currently I'm relying on a hack onto core Jekyll to achieve this:

# _plugins/set_posts.rb
Jekyll::Hooks.register :site, :post_read do |site|
  c = site.collections
  c['posts'].docs.concat c['news'].docs
  c['posts'].docs.concat c['planet'].docs
end

This way my feed.xml contains posts from all three collections. However as of Jekyll 4.3, jekyll doctor complains about multiple source files pointing to the same output file. It's absolutely right that I basically just duplicated all posts from desired collections into the _posts collection.

Now I can't find another good way to work around this, so I have to bring #289 up again. It's useful and hopefully easy to implement.
@iBug iBug linked a pull request Apr 4, 2023 that will close this issue
Open
@jekyllbot jekyllbot added the stale label Jun 4, 2023
@jekyllbot
Copy link
Contributor

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master/main branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

@0xb120
Copy link

0xb120 commented Jun 17, 2023

I would really appreciate this feature too!

@jekyllbot jekyllbot removed the stale label Jun 17, 2023
@jekyllbot
Copy link
Contributor

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master/main branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

@jekyllbot jekyllbot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 17, 2023
@parkr parkr reopened this Oct 17, 2023
@parkr parkr removed the stale label Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

cmd/doctor: Remove duplicate source paths from conflicting_urls check iBug-forks/jekyll
4 participants
@parkr @jekyllbot @iBug @0xb120