search

fedwiki / wiki-server

Federated Wiki client and server in Node.js
Other
153 stars 35 forks source link

adding sitemap.xml #114

Closed paul90 closed 8 years ago

paul90 commented 8 years ago

This pull request adds support for xml sitemaps

  1. change the sitemap generation to also save a sitemap.xml version
  2. add route to serve sitemap.xml
WardCunningham commented 8 years ago

I remember xml being mentioned in passing during a hangout but I don't remember what problem xml solved that wasn't solved more concisely by our convention to use json wherever possible. I don't see where any extra work is done to meet, say, the google recommendation for sitemaps. Am I missing something?

paul90 commented 8 years ago

xmlSitemapSave extacts the page-slug and date last modified from the sitemap to build sitemap.xml.

This is purely to help external search engines - they don't support JSON

opn commented 8 years ago

Cool. For Fedwiki would OPML not be better? Think this would give us a navigable index and give Dave Winer what he is looking for? On 9 Jan 2016 11:51 pm, "Ward Cunningham" notifications@github.com wrote:

Merged #114 https://github.com/fedwiki/wiki-server/pull/114.

— Reply to this email directly or view it on GitHub https://github.com/fedwiki/wiki-server/pull/114#event-510315446.