zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Remplacement de Gravatar par Jdenticon pour les avatars par défaut #6609

Closed Situphen closed 1 month ago

Situphen commented 7 months ago

PR liée au sujet sur le forum concernant l'intégration des sources externes

J'ai modifié les templates car Jdenticon utilise des éléments <canvas> ou <svg> au lieu de <img>. On peut utiliser cet outil pour personnaliser les avatars générés si on veut. Personnellement je trouve que les paramètres par défaut sont déjà très corrects !

Seul bémol avec ce remplacement : les membres actuels qui utilisaient Gravatar pour afficher leur avatar auront un avatar par défaut de Jdenticon. Ce que l'on peut faire pour éviter ça, c'est un script qui fait une requête vers Gravatar et si la réponse retourne un code 200 alors on ajoute l'URL de Gravatar en tant qu'avatar. Cela donnerait quelque chose comme ceci :

hash = md5(user.email.lower().encode("utf-8")).hexdigest()
gravatar_url = f"https://secure.gravatar.com/avatar/{hash}?d=404"
# Requête vers gravatar_url
# Si réponse 200
profile.avatar_url = gravatar_url
profile.save()
# Sinon, rien

QA :

coveralls commented 7 months ago

Coverage Status

coverage: 89.182% (-0.003%) from 89.185% when pulling 625c3f2cc6707493c4e77ac5dc2432e5564318ef on Situphen:gravatar into 92a76532b1a918ba7533ac3ccd442f6d1a0c45c5 on zestedesavoir:dev.

philippemilink commented 6 months ago

Ce que l'on peut faire pour éviter ça, c'est un script qui fait une requête vers Gravatar et si la réponse retourne un code 200 alors on ajoute l'URL de Gravatar en tant qu'avatar.

En réunion de dev's, on a confirmé que c'est ce qu'on voulait faire, a priori dans une migration.

Situphen commented 1 month ago

C'est prêt pour la QA ! J'ai ajouté une commande pour la migration des membres avec un Gravatar.

Situphen commented 1 month ago

Bien vu, c'est corrigé !

Situphen commented 1 month ago

C'est modifié !