Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correction de liens et images manquantes #135

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sylvinus
Copy link

Bonjour,

D'abord bravo pour le site statique bien conçu et plein d'infos !

Il semble que quelques liens et images se sont perdus au fil des migrations. Voici le procédé utilisé pour les corriger en masse :

  1. Convertir les URLs absolues en relatives. C'est une étape préparatoire pour que les étapes suivantes puissent détecter et corriger des URLs comme https://entrepreneur-interet-general.etalab.gouv.fr/defis/2019/iaflash.html.
  2. Faire tourner htmltest avec la configuration ci-dessous:
IgnoreAltMissing: true
CheckExternal: false
CheckDoctype: false
IgnoreAltEmpty: true
DirectoryPath: dist
IgnoreDirectoryMissingTrailingSlash: true
CheckMailto: false
  1. Corriger les centaines d'erreurs à coups de regexes ;)
  2. Importer des images manquantes depuis https://github.com/entrepreneur-interet-general/site-eig/tree/master/img
  3. Quelques corrections dans les layouts

Ce genre de corrections avait déjà été fait à plus petite échelle dans le site v2: entrepreneur-interet-general/site-eig-v2@e69573f

Je conseille d'inclure htmltest au build (il tourne en <1s sans faux positifs), et pourquoi pas à dashlord sur les autres sites statiques :)

@sylvinus sylvinus changed the title Fix 404s, correct links, import missing images from site-eig v1 Correction de liens et images manquantes Jul 29, 2024
@ericboucher
Copy link

Beau boulot @sylvinus ! Je plussoie. Je venais justement regarder ça après un être tombé sur un lien LinkedIn tout cassé :)

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.

2 participants