Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

HTML sitemap lists unpublished pages

Fechada Para iniciantes
  • Algo errado?
#883 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

@KrzysztofPajak já está trabalhando nisso.

Desde 7/10/2026.

  • #884 de @KrzysztofPajak — aberto

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
82/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
csharp
Domínio
backend

Direção de pesquisa

Comece pelo bloco de páginas em src/Web/Grand.Web/Features/Handlers/Common/GetSitemapHandler.cs e compare seus filtros com os do handler do sitemap XML em src/Business/Grand.Business.Messages/Commands/Handlers/Common/GetSitemapXMLCommandHandler.cs. Verifique o comportamento relacionado das páginas em Features/Handlers/Pages/GetPageBlockHandler.cs. O trabalho estará concluído quando /sitemap excluir páginas não publicadas e mantiver os filtros de sitemap e de data indicados; verifique o comportamento com os testes relevantes ou reproduzindo-o com uma página não publicada.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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

  1. In the admin panel create a page, tick "Include in sitemap" and leave "Published" unticked.
  2. Open /sitemap on the storefront.
  3. 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.

Linguagem predominante
C#
Estrelas
1.3k
Forks
549
Merge médio
11h 20min
PRs com merge (30d)
122

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de grandnode/grandnode2

Todas as issues de grandnode/grandnode2

Issues semelhantes

Mais issues de C#

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.