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

Add gatsby-plugin-feed to www #1569

Merged
merged 2 commits into from Jul 20, 2017
Merged

Add gatsby-plugin-feed to www #1569

merged 2 commits into from Jul 20, 2017

Conversation

secretfader
Copy link

@secretfader secretfader commented Jul 19, 2017

I'm not 100% on-board with defining logic in gatsby-config. What do you think?

Otherwise, the API exposed by the feed plugin makes this very simple to do.

@gatsbybot
Copy link
Collaborator

gatsbybot commented Jul 19, 2017

Deploy preview failed.

Built with commit 6a940c3

https://app.netlify.com/sites/using-drupal/deploys/5970eab7cf321c6cc955a7a9

@gatsbybot
Copy link
Collaborator

gatsbybot commented Jul 19, 2017

Deploy preview ready!

Built with commit 6a940c3

https://deploy-preview-1569--gatsbyjs.netlify.com

@secretfader
Copy link
Author

secretfader commented Jul 19, 2017

Looks like I have some cleanup to do: the feed doesn't validate.

Edit: It does validate, but we have some suggestions.

@KyleAMathews
Copy link
Contributor

It's not my favorite either but then I never have seen a pretty config on any real-world project.

The config could be moved to a separate module but I'm not sure that's any better

@secretfader
Copy link
Author

secretfader commented Jul 19, 2017

That's kinda where I'm at. My site has a config/ folder that keeps all this junk out, sorta. It's not that much better.

If you think this is good to go, I give it 👍 .

@KyleAMathews
Copy link
Contributor

KyleAMathews commented Jul 19, 2017

Deploy preview ready!

Built with commit 6a940c3

https://deploy-preview-1569--gatsbygram.netlify.com

@KyleAMathews KyleAMathews merged commit 40120b0 into gatsbyjs:master Jul 20, 2017
@secretfader secretfader deleted the feature/feed-www-blog branch July 20, 2017 17:51
@KyleAMathews
Copy link
Contributor

Looking good!

screen shot 2017-07-20 at 10 56 41 am

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.

None yet

3 participants