Conversation
- test suite for required sitemap elements
- generateSitemap writes sitemap to static folder
|
Would it be simpler for the sitemap to be generated dynamically? It's always up-to-date that way. |
I think I misread your 'attach it to a route' comment. Maybe you mean when a document becomes |
|
I meant that it's a dynamic route rather than a static file. So something like |
Oh ok that's a better idea. |
|
The xml format may be OK, but you can also just use a simple text file with each url separated by newlines, e.g. Maybe the xml format has some other advantages, but the text format is really simple. |
|
The robots.txt should be included in this PR to activate the site map. See #842 (comment) I think that this should be enough to start: |
|
Note that the sitemap should also include the homepage: |
- Add home route - tests update
| return loadDoc( docOpts ).then( getDocJson ); | ||
| })); | ||
| } ) | ||
| .then( docs2Sitemap ) |
There was a problem hiding this comment.
Maybe append the home page here?
There was a problem hiding this comment.
It's already appended in the sitemap.js code. Isn't that OK?
sitemapexposinggenerateSitemap( docs ):xml-jsto generate the xml string/document/sitemap?apiKey=xyzgenerateSitemapSample out
Refs #842