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

404 page #165

Merged
merged 9 commits into from
May 18, 2021
Merged

404 page #165

merged 9 commits into from
May 18, 2021

Conversation

fcamblor
Copy link
Collaborator

@fcamblor fcamblor commented May 7, 2021

Fix #12

Pour pas oublier... cf #12

image

@Floby
Copy link
Collaborator

Floby commented May 7, 2021

j'ai pas compris quand/comment cette page doit s'afficher.

@fcamblor
Copy link
Collaborator Author

fcamblor commented May 7, 2021

Alors j'ai pas très bien compris non plus, mais il semblerait que Google fasse des tests pour voir si on a des pages 404 (en appelant une URL random ?) et en l'occurence, il s'apercevrait qu'une page censée ne pas exister contient exactement le même contenu que l'index.

Bon, je t'avoue que ça me paraît assez capilotracté (mais c'est rigolo de faire une 404 alors...)
Pour l'historique, ça venait d'une préco de jbleprado sur Mattermost :

  • pour le 404, faudrait en créer une en effet et éviter de rediriger vers la home
  • pour la page index, il faut indiquer à google quelle page est "canonique" pour éviter que le crawler google crawl des pages similaires : https://developers.google.com/search/docs/advanced/crawling/consolidate-duplicate-urls
  • concernant, le sensible à la casse, même idée qu'au dessus, l'idée est d'avoir des URLs "unique" sinon il y a un risque de duplicated content, et donc le crawler google risque de moins crawler l'URL qu'il choisit représentative
  • concernant www et non www, meme idée, il faut indiquer à google quel domain crawler avec rel=canonical ; je pense que la redirection 301 de www.covidtracker.fr vers covidtracker.fr fonctionne (on ne parle pas ici de www.vitemadose.covidtracker.fr)

@Floby
Copy link
Collaborator

Floby commented May 8, 2021

pas sûr que ça reste valable pour un site généré côté front tout ça.
En revanche on pourrait relativement aisément générer un sitemap

@Floby
Copy link
Collaborator

Floby commented May 8, 2021

mais c'est vrai que c'est rigolo de faire une 404

@nhumblot
Copy link
Collaborator

nhumblot commented May 10, 2021

J'ai branché la page tout en la laissant statique, pas besoin de bouger ça dans un composant ? Je vois pas trop l'intérêt, je ne sais pas si c'est top de créer un slot not found dans l'index pour le SEO, ça ne risquerait pas de nous faire plus de mal qu'autre chose si c'est scrappé par un robot tel quel ? 🤔

Si vous préférez dans un composant, pas de soucis. 🙂

<meta http-equiv="X-UA-Compatible" content="ie=edge">
<title>404 - Page introuvable</title>
<script type="text/javascript">
document.write("<base href=\"" + document.location.protocol + "//" + document.location.host + "/\" />");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A quoi sert ce bout de code ?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est pour déterminer l'url de base du document pour les liens relatifs : https://developer.mozilla.org/fr/docs/Web/HTML/Element/base.

On reprend la racine du domaine au lieu de repartir de la page 404.html, ça permet ligne 123 d'avoir un retour à l'index avec un <a href=""> au lieu d'avoir à mentionner le index.html. La racine du domaine est récupérée dynamiquement pour que ça fonctionne sur d'autres environnements que la production.

<meta name="viewport"
content="width=device-width, user-scalable=no, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
<meta http-equiv="X-UA-Compatible" content="ie=edge">
<title>404 - Page introuvable</title>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je me dis que ça peut être sympa d'ajouter quand même les favicons sur cette page (et éventuellement les autres meta tags).

    <link rel="apple-touch-icon" sizes="180x180" href="/assets/images/favicon/apple-touch-icon.png?v=oLnE8zppbY">
    <link rel="icon" type="image/png" sizes="32x32" href="/assets/images/favicon/favicon-32x32.png?v=oLnE8zppbY">
    <link rel="icon" type="image/png" sizes="16x16" href="/assets/images/favicon/favicon-16x16.png?v=oLnE8zppbY">
    <link rel="mask-icon" href="/assets/images/favicon/safari-pinned-tab.svg?v=oLnE8zppbY" color="#5561d9">
    <link rel="shortcut icon" href="/assets/images/favicon/favicon.ico?v=oLnE8zppbY">

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrigé, merci pour le signalement. 🙂

@Luwangel
Copy link
Collaborator

@fcamblor @Floby @nhumblot J'ai ajouté deux petits commits pour que Google n'indexe pas la page. J'ai aussi rebase sur dev (j'avais un conflit sur un commit de suppression de paramètres inutiles qui avaient déjà été supprimés). Après ça, je crois qu'on est bon pour cette PR :D

@fcamblor fcamblor merged commit 88e4eb9 into dev May 18, 2021
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.

Pas de page 404
5 participants