MohammedThowfiq opened a new pull request, #3309:
URL: https://github.com/apache/sedona/pull/3309

   ## Did you read the Contributor Guide?
   
   Yes, I have read the Contributor Rules and Contributor Development Guide.
   
   ## Is this PR related to a ticket?
   
   Related to #3004. This addresses the `/latest/zh/sitemap.xml` 404 shown in 
the issue's screenshot.
   
   ## What changes were proposed in this PR?
   
   Material resolves `sitemap.xml` against each alternate language URL. The 
homepage advertises `/latest/zh/`, but mkdocs-static-i18n writes a combined 
sitemap only at the site root, leaving that request without a file.
   
   Add a post-build hook that exposes the combined XML and compressed sitemap 
under each enabled, non-default language root. Run it after i18n finishes its 
nested builds so the copies include all languages. The existing canonical URLs 
and root sitemap are preserved.
   
   Add a small multilingual build regression test and run it in the 
documentation workflow. The test covers unversioned, `latest/`, and 
`latest-snapshot/` site URLs, XML contents, gzip contents, and disabled/default 
locales.
   
   Scope: this fixes language-root sitemap requests. Material's separate 
handling of contextual alternate links on nested pages is unchanged.
   
   ## How was this patch tested?
   
   - Confirmed the live `/latest/zh/sitemap.xml` returns 404 while 
`/latest/sitemap.xml` returns 200.
   - Full baseline MkDocs build reproduced the missing Chinese sitemap.
   - All three regression scenarios failed without the hook and passed with it.
   - Full fixed MkDocs build passed; the Chinese XML and gzip copies match the 
combined sitemap (1966 URLs). The build emits existing warnings about 
unavailable generated Java/Scala/R/Python API pages and translation links; 
those API documentation builds were not run locally.
   - YAML lint, Black formatting, codespell and `git diff --check` passed for 
the changed files.
   
   ## Did this PR include necessary documentation updates?
   
   No public API changes. The new hook and test document their purpose and how 
to run the checks.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to