[Bug]: Lien vers le badge
vvatelot opened this issue · 8 comments
Que s'est il passé ?
Le lien vers le badge n'est pas correct (en tout cas sur la page de résultat...)
https://www.ecoindex.fr/resultat/?id=cc05241a-21a0-4006-868a-6d2ccc2ccb58
Sur quel navigateur avez vous rencontré ce problème ?
No response
Sur quel device rencontrez vous le problème ?
No response
Un message d'erreur est il apparu ?
No response
Je viens de regarder l'historique des versions, et la section qui parle du badge a été introduite sur la premier tag 0.17 (0.17.0rc1).
Le lien a toujours été tel quel depuis, aucune version plus récente n'ayant corrigé cela.
Quelle forme doit avoir le lien ? C'est une nouvelle feature de proposer un lien vers le badge qui représente le score de l'URL analysée ? Ou le lien doit simplement mener vers le repo du générateur de badge (ce qui serait très curieux)
Tiens, je n'avais pas vu que ça avait été introduit seulement depuis la dernière MEP...
Je dirais que dans l'immédiat on peut juste rediriger vers le projet du badge: https://github.com/cnumr/ecoindex_badge et voir par la suite si on intègre ça plus proprement directement dans le site..? Tu en penses quoi ?
Oui très bien. Comme ça on prend le temps de bien définir le besoin : extrait de code à copier ? bouton "copier le code" ? ...
En revoyant le code et en testant plusieurs URL, je pense que ce block a été pensé pour n'être affiché qu'en cas de note >= B.
Ce qui est logique compte-tenu du paragraphe suivant dans la page Confidentialité :
Le badge EcoIndex.fr peut être obtenu par n’importe quel·le utilisateur·ice à l’issue du test d’une URL en cas de l’obtention d’un score égale à A ou B
Par contre ce n'est pas du tout la logique en place actuellement : le paragraphe est affiché dans tous les cas.
Je serais finalement davantage d'avis de retirer complètement la section sur la page de résultats en attendant de clarifier le fonctionnement. Qu'en penses-tu @vvatelot ?
Je pense qu'il y a effectivement une petite mise à jour à faire au sein du collectif pour comprendre ce qui était la cible initiamelement et est ce que c'est toujours d'actualité ?... Je relance en interne... Mais pour l'instant, je dirais que oui, on supprime la section sur la page des résultats et voir pour proposer quelque chose de plus clean une fois que c'est validé
Perso, je trouve étrange de n'afficher le badge que pour un score de A ou B. Le badge devrait s'afficher quel que soit le score EcoIndex.
Ok, dans ce cas je pense qu'on peut remettre ce paragraphe dans le site en faisant le lien avec le projet https://github.com/cnumr/ecoindex_badge si tout le monde est ok