Skip to content

fix: pages Dev avec parenthèses affichées « Page Not Found » (#351) - #356

Open
ohugonnot wants to merge 2 commits into
mainfrom
351-pages-parentheses
Open

ohugonnot wants to merge 2 commits into
mainfrom
351-pages-parentheses

Conversation

@ohugonnot

Copy link
Copy Markdown
Contributor

Closes #351

Le problème

Trois pages de la partie Dev ont des parenthèses dans leur nom de fichier. Docusaurus les génère alors comme une page « Page Not Found », vide, dans toutes les langues. C'est le cas en ligne aujourd'hui :

/Dev/Introduction/development/Networking-(Network-Messages)/                       → Page Not Found
/Dev/Introduction/development/Navigating-DM-codebases-(like-tgstation)/            → Page Not Found
/Dev/Introduction/development/Proper-Spawning-and-Despawning-(Object-Lifecycle)/   → Page Not Found

Ce que fait la PR

  • Renomme les 3 fichiers sans parenthèses :
    • Networking-Network-Messages.md
    • Navigating-DM-codebases-like-tgstation.md
    • Proper-Spawning-and-Despawning-Object-Lifecycle.md
  • Met à jour les 4 liens qui pointaient vers ces pages (Mirror, Creating-Items-and-Objects ×2, Component-Development-Checklist). Le lien de Mirror pointait vers un ancien chemin /docs/development/... qui n'existait plus : il pointe maintenant vers la bonne page.
  • Ajoute un title: aux deux pages qui n'avaient pas de titre. Sans ça, le menu aurait affiché le nom du fichier (« Networking-Network-Messages »). Avec, il affiche « Networking (Network Messages) » et « Navigating DM codebases (like tgstation) ».

Les traductions ne sont pas concernées : aucune n'existe pour ces 3 pages.

Vérifié

  • Le build anglais réussit.
  • Le site est servi par busybox, comme en prod : les 3 pages répondent 200, avec leur contenu, leur titre et leur libellé dans le menu.
  • Le robot qui parcourt le site ne trouve aucun lien cassé vers ces pages.
  • npm run test:links compte un lien sans cible de moins en anglais.
  • Plus aucun fichier du dépôt n'a de parenthèses dans son nom, et plus rien ne pointe vers les anciens noms.

Hors de cette PR

  • Les autres liens cassés de ces pages : les antislashs sont traités par fix: liens à antislash et menu des métiers (#117) #350, et les anciens chemins /docs/development/... de Mirror font partie des liens sans cible, à corriger à part.
  • Les anciennes adresses avec parenthèses ne redirigent pas vers les nouvelles. Elles affichaient déjà une page vide, donc on ne perd rien.

@ohugonnot ohugonnot self-assigned this Oct 25, 2026
@ohugonnot
ohugonnot marked this pull request as ready for review October 25, 2026 12:54
@ohugonnot

Copy link
Copy Markdown
Contributor Author

Revue faite (2 relecteurs indépendants, aucun point relevé) et contrôle CI vert : je passe la PR en prête. Elle est indépendante des autres PR ouvertes (elle ne touche aucune ligne modifiée par #350 ou #353) et peut être fusionnée dans n'importe quel ordre.

@ohugonnot

Copy link
Copy Markdown
Contributor Author

Correction de mon message précédent : cette PR n'est pas totalement indépendante. Elle entre en conflit avec #350 sur deux lignes voisines de docs/5_Dev/Introduction/development/Component-Development-Checklist.md (#350 corrige l'antislash de la ligne 6, celle-ci le lien de la ligne 5). Le conflit est simple. Pour garder l'ordre déjà prévu, fusionnez #350 d'abord, puis je mettrai cette PR à jour sur main et je vous préviendrai.

This branch has not been deployed

No deployments
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.

3 pages Dev avec des parenthèses dans le nom s'affichent « Page Not Found »

1 participant