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

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

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Situphen
Copy link
Member

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 :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • Vérifier que les avatars s'affichent correctement partout sur le site (avatars par défaut ou personnalisés)

@Situphen Situphen added this to En développement in Suivi des PR via automation Apr 21, 2024
@coveralls
Copy link

Coverage Status

coverage: 88.75% (-0.002%) from 88.752%
when pulling bee8bb8 on Situphen:gravatar
into 7b70da3 on zestedesavoir:dev.

@philippemilink
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Suivi des PR
  
En développement
Development

Successfully merging this pull request may close these issues.

None yet

3 participants