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

Im/sitemap #4332

Merged
merged 4 commits into from
Nov 15, 2024
Merged

Im/sitemap #4332

merged 4 commits into from
Nov 15, 2024

Conversation

IanMadd
Copy link
Contributor

@IanMadd IanMadd commented Nov 15, 2024

Description

The main sitemap.xml now indexes other sitemaps.
It reads through the sitemaps list in the config/params.toml file and lists each sitemap included there.

A new sitemap-main.xml indexes all pages generated from chef-web-docs.

Definition of Done

Issues Resolved

https://progresssoftware.atlassian.net/browse/CHEF-16363

Related PRs

Check List

  • Spell Check
  • Local build
  • Examine the local build
  • All tests pass

Signed-off-by: Ian Maddaus <[email protected]>
@IanMadd IanMadd requested a review from a team as a code owner November 15, 2024 17:22
Copy link

netlify bot commented Nov 15, 2024

Deploy Preview for chef-web-docs ready!

Name Link
🔨 Latest commit 53b13db
🔍 Latest deploy log https://app.netlify.com/sites/chef-web-docs/deploys/6737c0eee7d4420008627de1
😎 Deploy Preview https://deploy-preview-4332--chef-web-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Ian Maddaus <[email protected]>
Signed-off-by: Ian Maddaus <[email protected]>
Copy link

sonarcloud bot commented Nov 15, 2024

@IanMadd IanMadd merged commit c51db49 into main Nov 15, 2024
10 of 12 checks passed
@IanMadd IanMadd deleted the im/sitemap branch November 15, 2024 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant