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

Améliore la page qui liste les membres utilisant la même IP #6545

Merged
merged 2 commits into from
Oct 23, 2023

Conversation

philippemilink
Copy link
Member

@philippemilink philippemilink commented Oct 15, 2023

  • Affichage à côté de l'adresse IP la localisation rapportée par GeoIP (comme ce qu'on peut voir sur la page de profil)
  • Utilisation d'un tableau pour liste les membres qui ont utilisé cette adresse IP, pour pouvoir facilement indiquer aussi la date d'inscription de ces membres
  • Affichage de Jamais si un membre a créé son compte avec cette adresse IP, mais ne s'est jamais connecté ensuite (jusqu'alors au lieu d'avoir une date, on n'avait rien, ce qui pouvait être un peu perturbant)
  • Création d'une fonction qui donne la localisation à partir d'une adresse IP, pour éviter de dupliquer ce code
  • Amélioration de l'ordre de certains import
  • Ajout de {% blocktrans %} autour d'un texte

Contrôle qualité

Il faut être membre de l'équipe de modération ou administrateur pour pouvoir QA cette PR.

Pour avoir les données GeoIP, le plus simple sera de la faire sur la bêta. Je vais déployer la PR sur la bêta, je mettrai à jour les instructions de QA ensuite.

  • Aller sur la page de profil d'un membre : la localisation doit bien s'afficher à côté de l'IP
  • Aller sur la page de l'adresse IP : un tableau de membres s'affiche, indiquant leur date d'inscription et de dernière connexion. S'il n'y a pas eu de de connexion depuis l'inscription, Jamais est affiché.
  • Avec une IPv6, on a la même chose, mais en plus pour tous les membres connectés avec l'adresse IP/64.

@philippemilink philippemilink added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 15, 2023
@coveralls
Copy link

coveralls commented Oct 15, 2023

Coverage Status

coverage: 88.803% (-0.02%) from 88.818% when pulling 120fbbc on philippemilink:improve-memberip into c29c53f on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

Petit commentaire mineur.

Sinon ça fonctionne bien d'après mes tests.

zds/member/views/moderation.py Outdated Show resolved Hide resolved
@philippemilink
Copy link
Member Author

J'ai déployé sur la bêta, corrigé un bug au passage, pris en compte ta suggestion et mis à jour les instructions de QA avec les liens pour tester sur la bêta.

@Arnaud-D Arnaud-D dismissed their stale review October 17, 2023 12:49

Commentaire pris en compte

@Arnaud-D
Copy link
Contributor

Je n'ai pas les droits pour QA sur la bêta. On me le rajoute, on trouve un autre dév qui les a ou on demande à un staff de la faire ?

@philippemilink
Copy link
Member Author

Je viens de te rajouter les droits modération et staff, ça devrait être suffisant je pense.

Arnaud-D
Arnaud-D previously approved these changes Oct 22, 2023
Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

QA OK, avec un commentaire très mineur.

templates/member/admin/memberip.html Show resolved Hide resolved
Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

QA OK ✔️

C'est impeccable comme ça.

templates/member/admin/memberip.html Show resolved Hide resolved
@Arnaud-D Arnaud-D merged commit 0a733a7 into zestedesavoir:dev Oct 23, 2023
12 checks passed
@philippemilink philippemilink deleted the improve-memberip branch October 23, 2023 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants