HTML sitemap lists unpublished pages
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
Línea de trabajo
Empieza con el bloque de páginas en src/Web/Grand.Web/Features/Handlers/Common/GetSitemapHandler.cs y compara sus filtros con los del controlador del sitemap XML en src/Business/Grand.Business.Messages/Commands/Handlers/Common/GetSitemapXMLCommandHandler.cs. Comprueba el comportamiento relacionado de las páginas en Features/Handlers/Pages/GetPageBlockHandler.cs. El trabajo estará terminado cuando /sitemap excluya las páginas no publicadas y mantenga los filtros de sitemap y fecha indicados; verifica el comportamiento con las pruebas pertinentes o reproduciéndolo con una página no publicada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happens
The storefront HTML sitemap (/sitemap) lists CMS pages that are not published. The title of an unpublished page is shown publicly, and its link leads nowhere because the page itself checks Published and does not render.
Steps to reproduce
- In the admin panel create a page, tick "Include in sitemap" and leave "Published" unticked.
- Open
/sitemapon the storefront. - The page is listed under the pages section; following the link does not show the page.
Cause
src/Web/Grand.Web/Features/Handlers/Common/GetSitemapHandler.cs (pages block, around lines 110-114) takes pages from IPageService.GetAllPages(storeId), which does not filter on Published, and then filters only on IncludeInSitemap and the start/end dates.
The XML sitemap already does this correctly: src/Business/Grand.Business.Messages/Commands/Handlers/Common/GetSitemapXMLCommandHandler.cs (around line 332) filters on t.Published && t.IncludeInSitemap && .... The page view enforces Published in Features/Handlers/Pages/GetPageBlockHandler.cs.
Expected behaviour
Only published pages with "Include in sitemap" ticked (and within their start/end dates) appear on /sitemap, consistent with the XML sitemap.
- Lenguaje dominante
- C#
- Estrellas
- 1.3k
- Forks
- 549
- Merge medio
- 7 h 58 min
- PR fusionados (30 d)
- 52
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de grandnode/grandnode2
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
grandnode/grandnode2#881 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
grandnode/grandnode2#880 ·
Los mantenedores suelen responder en 1 día
-
Please add the offer systemAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
grandnode/grandnode2#849 ·
Los mantenedores suelen responder en 1 día
-
grandnode/grandnode2#499 ·
Los mantenedores suelen responder en 1 día
-
grandnode/grandnode2#410 ·
Los mantenedores suelen responder en 1 día
Todos los issues de grandnode/grandnode2
Issues similares
-
Bug pulumi/pulumi
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
activescott/lessmsi#306 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
[Bug]: `winapp ui <command> --on sandbox --help` starts sandbox setup instead of showing helpAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Python: Bug: split_plaintext_paragraph / split_markdown_paragraph can return a chunk larger than max_tokensPosiblemente ocupada @xThreeh la tomó hoy. Abiertopython triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
microsoft/semantic-kernel#14566 ·
Los mantenedores suelen responder en 4 días