Skip to content

Add sitemap.xml to show_page_in_sitemap translation#3744

Open
antwertinger wants to merge 1 commit intoAlchemyCMS:mainfrom
foureb:update-sitemap-translation
Open

Add sitemap.xml to show_page_in_sitemap translation#3744
antwertinger wants to merge 1 commit intoAlchemyCMS:mainfrom
foureb:update-sitemap-translation

Conversation

@antwertinger
Copy link
Contributor

What is this pull request for?

Clarify show_page_in_sitemap label to reference sitemap.xml explicitly

The previous translations for the show_page_in_sitemap checkbox label were vague (e.g. "Show page in sitemap"). It was not immediately clear to users what file or system was affected.

By including sitemap.xml literally, the label now clearly communicates that checking this option controls whether the page appears in the sitemap.xml file — making the checkbox's purpose unambiguous regardless of the user's language.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@antwertinger antwertinger requested a review from a team as a code owner March 13, 2026 16:27
@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.47%. Comparing base (09b9beb) to head (5d16186).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3744   +/-   ##
=======================================
  Coverage   97.47%   97.47%           
=======================================
  Files         317      317           
  Lines        8428     8428           
=======================================
  Hits         8215     8215           
  Misses        213      213           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use the term Google uses in their docs

show_eq: "Show EQ"
show_navigation: "Show in navigation"
show_page_in_sitemap: "Show page in sitemap."
show_page_in_sitemap: "Show page in sitemap.xml."
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sitemap.xml is a very technical term. I would like to avoid it.

Google Search Console and most SEO Tools use the term "XML Sitemap", so it should be widely known.

Suggested change
show_page_in_sitemap: "Show page in sitemap.xml."
show_page_in_sitemap: "Show Page in XML Sitemap"

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