Skip to content
This repository has been archived by the owner on Sep 7, 2020. It is now read-only.

Add sitemap webpack plugin #907

Merged
merged 1 commit into from
Dec 16, 2016
Merged

Add sitemap webpack plugin #907

merged 1 commit into from
Dec 16, 2016

Conversation

cedricdelpoux
Copy link
Contributor

fix #621


The ``phenomicLoader`` allows you to generate sitemap very easily.

The phenomic-theme-base already handles a default RSS sitemap that will grab all your urls
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to sitemap's README, it generates a XML sitemap, not a RSS one.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

of course, my bad

@cedricdelpoux cedricdelpoux force-pushed the sitemap-webpack-plugin branch 2 times, most recently from 1cd51ba to 8c17437 Compare December 6, 2016 08:07
@@ -125,6 +125,7 @@
"react-router": "^2.3.0",
"react-test-renderer": "^15.3.2",
"redux": "^3.0.0",
"sitemap": "~1.8.2",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you use ^ instead of ~ like other package?
Also can you use yarn so the lock file is updated?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. But the yarn.lock is not under version. So what is the point to run it locally ?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah you are right, I removed it :D but will add it back.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok perfect

@cedricdelpoux
Copy link
Contributor Author

anything I can do to move on ?

@@ -125,6 +125,7 @@
"react-router": "^2.3.0",
"react-test-renderer": "^15.3.2",
"redux": "^3.0.0",
"sitemap": "^1.8.2",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be in deps, not devDeps.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Sitemap is generated during build so I believed package used for this were in dev deps. But I saw the rss dep was in deps too

Copy link
Owner

@MoOx MoOx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be my last comments!

@@ -0,0 +1,51 @@
---
title: How to use generate sitemap in Phenomic
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How to generate a sitemap with Phenomic?

title: How to use generate sitemap in Phenomic
---

The ``phenomicLoader`` allows you to generate sitemap very easily.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Phenomic allows you to generate a sitemap very easily if you use the phenomicLoader.


The ``phenomicLoader`` allows you to generate sitemap very easily.

The phenomic-theme-base already handles a default XML sitemap that will grab all your urls
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add a . at the end of the line please.

@cedricdelpoux
Copy link
Contributor Author

Thank you for your feedback. I corrected last typos

@MoOx MoOx merged commit b9180bd into master Dec 16, 2016
@MoOx MoOx deleted the sitemap-webpack-plugin branch December 16, 2016 07:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generating a sitemap
3 participants