Project

General

Profile

Development #44663

suggestion des domaines communs des courriels: proposer une option du champ pour activation de la fonctionnalité

Added by Serghei Mihai 5 days ago. Updated 3 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
01 Jul 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Cela semble être génant dans certains cas (exemple: #44659)
L'option devrait être désactivée par défaut.

0001-emails-allow-configuring-known-valid-domains-for-typ.patch View (6.71 KB) Frédéric Péters, 03 Jul 2020 08:05 AM

Associated revisions

Revision a95f583c (diff)
Added by Frédéric Péters 3 days ago

emails: allow configuring known/valid domains for typo fixes (#44663)

History

#1 Updated by Serghei Mihai 5 days ago

  • Subject changed from autocompletion des courriels: proposer une option du champ pour activation de la fonctionnalité to suggestion des domaines communs des courriels: proposer une option du champ pour activation de la fonctionnalité

#3 Updated by Frédéric Péters 5 days ago

Nope. Mais rendre possible la configuration d’éléments supplémentaires pour la liste de domaines connus mais non suggérés, sûr, go.

#4 Updated by Benjamin Dauvergne 5 days ago

C'est peut-être juste que le test est trop sensible.

>> "laposte.fr".similarity("lametro.fr")
0.81
>> "gmail.com".similarity("yopmail.com")
0.872053872053872

Un seuil de 0.9 voir un peu plus serait peut-être suffisant.

#5 Updated by Thomas Noël 5 days ago

Yep faut essayer le 0.9 (voir si ça continuer à bien proposer bien des choses sur hoomail.fr), sinon j'imaginais ça comme plan pour permettre d'ajouter des domaines à suggérer / à ignorer :

  • avoir dans site-options.cfg des variables :
    • email_suggestion_more_known_domains qui s'ajoutera à known_domains
    • email_suggestion_more_well_known_domains qui s'ajoutera à well_known_domains
  • faire une vue /email_suggestion.js qui contiendra WCS_EMAIL_SUGGESTION.more_known_domains et WCS_EMAIL_SUGGESTION.more_well_known_domains à partir de ces deux variables
  • ajouter ../../email_suggestion.js quand on utilise qommon.forms.js (de la même façon qu'on ajoute ../../i18n.js)
  • utiliser WCS_EMAIL_SUGGESTION.more_* dans qommon.forms.js pour compléter les known_domains et well_known_domains

C'est l'idée ?

(Ensuite, dans hobo, ajouter la config de ces deux variables dans l'espace "emails")

#6 Updated by Benjamin Dauvergne 5 days ago

Pour hoomail ça passe même à 0.95 :

"hoomail.com".similarity("hotmail.com")
0.9515151515151515

On trouvera pas au pif, faut faire une liste de cas connus et voir les chiffres.

#7 Updated by Benjamin Dauvergne 5 days ago

Thomas Noël a écrit :

  • avoir dans site-options.cfg des variables :
    • email_suggestion_more_known_domains qui s'ajoutera à known_domains
    • email_suggestion_more_well_known_domains qui s'ajoutera à well_known_domains

Pourquoi deux listes ? Y en a qu'une dans le code actuel.

  • faire une vue /email_suggestion.js qui contiendra WCS_EMAIL_SUGGESTION.more_known_domains et WCS_EMAIL_SUGGESTION.more_well_known_domains à partir de ces deux variables

Mouais ça pourrait aller directement dans le code du EmailWidget dans un data-well-known-domains="...."

#8 Updated by Frédéric Péters 5 days ago

Pourquoi deux listes ? Y en a qu'une dans le code actuel.

Il y en a deux.

Faire ça via l'écran de configuration /backoffice/settings/emails/options, je suis pour arrêter de taper tout en site-options.cfg.

#9 Updated by Frédéric Péters 3 days ago

#10 Updated by Serghei Mihai 3 days ago

  • Status changed from Solution proposée to Solution validée
  • Assignee set to Frédéric Péters

#11 Updated by Frédéric Péters 3 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit a95f583c3e492800a7458f245b475e9cffca1d93
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Jul 3 08:04:21 2020 +0200

    emails: allow configuring known/valid domains for typo fixes (#44663)

#12 Updated by Frédéric Péters 3 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF