Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

HTML sitemap lists unpublished pages

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#883 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@KrzysztofPajak y travaille déjà.

Depuis le 7/10/2026.

  • #884 par @KrzysztofPajak — ouverte

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
82/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
csharp
Domaine
backend

Piste de recherche

Commencez par le bloc des pages dans src/Web/Grand.Web/Features/Handlers/Common/GetSitemapHandler.cs et comparez ses filtres à ceux du gestionnaire de sitemap XML dans src/Business/Grand.Business.Messages/Commands/Handlers/Common/GetSitemapXMLCommandHandler.cs. Vérifiez le comportement associé aux pages dans Features/Handlers/Pages/GetPageBlockHandler.cs. Le travail est terminé lorsque /sitemap exclut les pages non publiées tout en conservant les filtres de sitemap et de date indiqués ; vérifiez le comportement à l’aide des tests pertinents ou en le reproduisant avec une page non publiée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
C#
Étoiles
1.3k
Forks
549
Merge moyen
7 h 58 min
PR mergées (30 j)
52

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de grandnode/grandnode2

Toutes les issues de grandnode/grandnode2

Issues similaires

Plus d'issues C#

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.