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

You get a feed! You get a feed! Feeds for everyone! #182

Closed
wants to merge 4 commits into from
Closed

You get a feed! You get a feed! Feeds for everyone! #182

wants to merge 4 commits into from

Conversation

dgreenley
Copy link
Contributor

This is the initial work for feed generation via Stackgen.rb. NOTE: The new feeds should be functional and are ready to test, but they haven't been inserted into any pages as yet! (This pull request is really so you know I'm poking at #61.)

I've moved the code for the feeds themselves into a layout, with the editable portions like the feed image, email addresses, podcast title, etc. moved into the front matter so small edits are less daunting. This covers the three feed types: the main feed of all posts, a feed for each tag (except for the 'podcasts' tag special case) and the podcast feed, which is podcasts tag with all the lovely iTunes information, at the correct podcast feed location (/feed/podcast/ not /feed/podcasts/). Podcast download enclosures are included in all feeds should episodes make a guest appearance anywhere else.

I went with a feed for every tag, for now, since that covers everything including the main channels. It may be overkill but who am I to judge what people want to subscribe to? (It does lead to both announcement and announcements, though? Don't blame me!)

@JonHMChan
Copy link
Contributor

Apologies for the delay on this - will test internally and see how to push.

@dgreenley
Copy link
Contributor Author

No worries, they work in my testing, but I don't consider this one quite ready for prime time--still thinking about maybe author feeds too? Maybe "everything that gets paginated gets a feed"? Also, inserting them into the appropriate page headers.

However, any additional testing on this is all good (:

@dgreenley dgreenley closed this Sep 16, 2017
@dgreenley dgreenley deleted the feed-generator branch September 16, 2017 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants