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.

Sign up for GitHub

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

Jump to bottom

gatsby-plugin-sitemap: incorrect sitemap path in index sitemap #31167

Closed
moonmeister opened this issue May 1, 2021 · 2 comments · Fixed by #31184
Closed

gatsby-plugin-sitemap: incorrect sitemap path in index sitemap #31167

moonmeister opened this issue May 1, 2021 · 2 comments · Fixed by #31184
Assignees
@moonmeister
Labels
type: bug An issue or pull request relating to a bug in Gatsby type: upstream Issues outside of Gatsby's control, caused by dependencies

Comments

@moonmeister
Copy link
Contributor

moonmeister commented May 1, 2021 •
edited
Loading

Description

The site map index doesn't include the sub-folder that the sitemap files are placed in.

The current work around would be to set the output config to /.
#31130

This issue has been reported upstream at ekalinin/sitemap.js#359

Steps to reproduce

see the up stream issue for more info. This is being opened to track that issue on the gatsby side so our dependencies can be updated when that issue is fixed.

Environment

gatsby-plugin-sitemap: ^4.0
@moonmeister moonmeister added type: bug An issue or pull request relating to a bug in Gatsby type: upstream Issues outside of Gatsby's control, caused by dependencies topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label labels May 1, 2021
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 1, 2021
@moonmeister moonmeister mentioned this issue May 1, 2021
Closed
@moonmeister moonmeister added topic: sitemap and removed topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label labels May 1, 2021
@moonmeister moonmeister mentioned this issue May 1, 2021
Closed
@moonmeister
Copy link
Contributor Author

I'm working on a fix for this

@moonmeister moonmeister self-assigned this May 2, 2021
@moonmeister moonmeister removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 2, 2021
@moonmeister moonmeister mentioned this issue May 2, 2021
Merged
@LekoArts LekoArts mentioned this issue May 4, 2021
Closed
@alexgladd
Copy link

Note that until PR #31130 is deployed, a better workaround is setting the output config to /./. Otherwise, the sitemap href in the link tag doesn't work.

@rudouglas rudouglas mentioned this issue May 4, 2021
Closed
@ichik ichik mentioned this issue May 6, 2021
Closed
@gehaktmolen gehaktmolen mentioned this issue May 11, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@moonmeister moonmeister

Labels
type: bug An issue or pull request relating to a bug in Gatsby type: upstream Issues outside of Gatsby's control, caused by dependencies
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(gatsby-plugin-sitemap): Sitemap path bug moonmeister/gatsby
2 participants
@moonmeister @alexgladd