Skip to content

remove trailing .htmls #60

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

Closed
wants to merge 3 commits into from
Closed

remove trailing .htmls #60

wants to merge 3 commits into from

Conversation

sanscontext
Copy link
Contributor

Signed-off-by: sanscontext lrubin@segment.com

Signed-off-by: sanscontext <lrubin@segment.com>
@sanscontext
Copy link
Contributor Author

We can probably ditch the logic to parse seo_changefreq and seo-priority now because I set these with defaults in _config.yml.

@XavierAgostini
Copy link
Contributor

@sanscontext i'm not seeing trailing HTML references when i generate the sitemap

@sanscontext
Copy link
Contributor Author

@XavierAgostini Correct, you shouldn't be seeing them. That's one of the improvements in this PR.

@XavierAgostini
Copy link
Contributor

To clarify when you generate the sitemap on the master branch there are no trailing 'html' references in the site map.

@sanscontext
Copy link
Contributor Author

@XavierAgostini can we verify that we're still generating the sitemap using the sitemap.xml liquid template then?

@XavierAgostini
Copy link
Contributor

correct we are

@sanscontext sanscontext deleted the fix-sitemap branch December 3, 2019 18:30
bot-docsteam added a commit that referenced this pull request Oct 13, 2021
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