Development #44663
suggestion des domaines communs des courriels: proposer une option du champ pour activation de la fonctionnalité
0%
Description
Cela semble être génant dans certains cas (exemple: #44659)
L'option devrait être désactivée par défaut.
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a presque 4 ans
- Sujet changé de autocompletion des courriels: proposer une option du champ pour activation de la fonctionnalité à suggestion des domaines communs des courriels: proposer une option du champ pour activation de la fonctionnalité
Mis à jour par Frédéric Péters il y a presque 4 ans
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.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
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.
Mis à jour par Thomas Noël il y a presque 4 ans
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")
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
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.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
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="...."
Mis à jour par Frédéric Péters il y a presque 4 ans
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.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-emails-allow-configuring-known-valid-domains-for-typ.patch 0001-emails-allow-configuring-known-valid-domains-for-typ.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Frédéric Péters
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution validée à 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)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
emails: allow configuring known/valid domains for typo fixes (#44663)