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 sitemap configuration to configuration file #7

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

darklinkpower
Copy link
Contributor

Just a small update. I just tried searching on google for something in the manual and for some reason it's linking to the old locations:

image

From a quick search I stumbled upon this and the reason could be that the docs lack a sitemap.

This PR helps to generate the sitemap while building the site, I don't know if it will fix the issue of Google not picking it up but it won't hurt and I think it's good practice to have one.


Example of generate sitemap.xml file:

<?xml version="1.0" encoding="utf-8"?>
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
  <url>
    <loc>https://api.playnite.link/docs/api/index.html</loc>
    <lastmod>2023-10-09T05:52:39-06:00</lastmod>
    <changefreq>daily</changefreq>
    <priority>0.5</priority>
  </url>
  <url>
    <loc>https://api.playnite.link/docs/changelog.html</loc>
    <lastmod>2023-10-09T05:52:39-06:00</lastmod>
    <changefreq>daily</changefreq>
    <priority>0.5</priority>
  </url>
   ...

@JosefNemec
Copy link
Owner

It's because Google is really slow re-indexing. Not sure if sitemap will help much. Manual wasn't indexed at all even couple weeks ago, I had to explicitly request the site to be indexed (Bing for example had no issue doing it automatically).

@JosefNemec JosefNemec merged commit 2aac513 into JosefNemec:main Oct 9, 2023
@darklinkpower
Copy link
Contributor Author

I see, fingers crossed this helps by some luck.

@darklinkpower
Copy link
Contributor Author

Good news, this seems to have fixed the docs not being indexed by Google. I'm surprised by how fast this was fixed.

image

image

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