Sur les stats de courriels, le pourcentage d'actifs est réel : le champ est obligatoire.
Sur les stats XMPP et Mastodon : le champ n'est pas forcément renseigné, rajouter un pourcentage % total renseigné disponible au-dessus
Actuellement, on a :
Domaines XMPP
Sur 2046 comptes utilisés sur le site au cours des trois derniers mois :
Domaines à plus de trois comptes | Nombre de comptes | Actifs |
---|---|---|
jabber.fr | 40 | 2% |
gmail.com | 20 | 1% |
jabber.org | 13 | 1% |
im.apinc.org | 12 | 1% |
chapril.org | 5 | 0% |
movim.eu | 4 | 0% |
=> le pourcentage est calculé par rapport à 2046 comptes utilisés alors qu'il n'y a que 174 comptes XMPP renseignés (et 119 mastodon)
L'affichage serait (le titre « renseignés » peut être changé pour un meilleur terme) :
Sur 2046 comptes utilisés sur le site au cours des trois derniers mois, pour 174 champs XMPP renseignés :
Domaines à plus de trois comptes | Nombre de comptes | Actifs | renseignés |
---|---|---|---|
jabber.fr | 40 | 2% | 23% |
gmail.com | 20 | 1% | 11% |
jabber.org | 13 | 1% | 7% |
im.apinc.org | 12 | 1% | 7% |
chapril.org | 5 | 0% | 3% |
movim.eu | 4 | 0% | 2% |
Le code à modifier est dans :
- https://github.com/linuxfrorg/linuxfr.org/blob/master/app/views/statistics/users.html.haml pour l'affichage
- https://github.com/linuxfrorg/linuxfr.org/blob/master/app/models/statistics/users.rb pour les données disponibles
RàF : trouver où est défini #{@stats.pctrecent(domain["cnt"])}
pour rajouter un calcul tenant compte de @stats.filled("jabber_id")
, idem pour @stats.filled("mastodon_url")
=> ok, c'est dans models/statistics/users.rb :
def pctrecent(value)
"%.0f%%" % (100.0 * value / nb_recently_seen_accounts)
end
donc soit ajouter fonction au modèle (le @stats.
n'est sans doute pas nécessaire)
def pctrecent_filled(value,field)
"%.0f%%" % (100.0 * value / @stats.filled(field))
end
et l'utiliser dans la vue pour ajouter la colonne
%td
#{@stats.pctrecent_filled(domain["cnt"],"jabber_id")}
soit faire le calcul directement dans la vue
l'idée est de proposer le patch afférent (à fournir en lien sur github et proposer une PR)
# pull request proposée
Posté par BAud (site web personnel) . Évalué à 2 (+0/-0).
voici une proposition de pull-request correspondant à l'évolution demandée :
https://github.com/linuxfrorg/linuxfr.org/pull/397
pour l'instant, j'ai choisi d'ajouter une fonction au modèle tenant compte de ceux ayant rempli le champs xmpp / mastodon (ayant un jabber_id / mastodon_url déclaré).
Je puis aussi proposer un patch pour tout calculer dans la vue, si c'est plus lisible.
Envoyer un commentaire
Suivre le flux des commentaires
Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.