demarches-simplifiees / demarches-simplifiees.fr

Dématérialiser et simplifier les démarches administratives
https://www.demarches-simplifiees.fr
GNU Affero General Public License v3.0
193 stars 88 forks source link

ETQ administrateur, lorsque je gère ma liste d'expert invités sur une démarche, je suis guidé pour eviter les typos #10524

Closed mfo closed 3 months ago

mfo commented 3 months ago

follows_up : https://github.com/demarches-simplifiees/demarches-simplifiees.fr/pull/10499 cf: https://github.com/demarches-simplifiees/demarches-simplifiees.fr/issues/10418

les saisies en batch cherchent les typos dans les batchs

problème, COI

le contexte global est que : en gros, sur l'internet mondial il y a trop de spam et des gens mécontant. les services devraient garantir/valider que les mails saisis sont bien les personnes a qui le mail saisi appartient de sorte a ne pas spammer un voisin en saisissant son mail sur un site.

solution,

on a deja une 'solution' pr eviter les typos quand on s'inscrit sur DS (création de compte).

sauf qu'on a aussi des retours d'instructeurs/expert qui nous disent avoir mis des mails ayant des typos dans les comptes d'instructeurs et experts qu'ils ont eux même invités. en parallel, un des "gardiens" de l'antispam de l'internet mondial nous dit qu'on est des spammeurs.

Aujourd'hui on cherche donc a faire des efforts sur ce sujet. Ici on a donc un petit pas : quand un admin saisi un mail, on tente de détecter des typos parmis les noms de domaines que nous connaissons. L'idée est de :

  1. remonter l'erreur
  2. proposer un choix (typo ou pas typo ?)
  3. effectuer un choix

resultat (validé par Marlène)

Capture d’écran 2024-06-20 à 3 28 03 PM
codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 97.14286% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.09%. Comparing base (9cbf1d3) to head (77bf93d). Report is 28 commits behind head on main.

Files Patch % Lines
...s/administrateurs/experts_procedures_controller.rb 95.23% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10524 +/- ## ========================================== + Coverage 81.08% 81.09% +0.01% ========================================== Files 1228 1230 +2 Lines 26002 26049 +47 Branches 4701 4713 +12 ========================================== + Hits 21084 21125 +41 - Misses 4918 4924 +6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

LeSim commented 3 months ago

top.

Screencast from 2024-06-24 10-14-35.webm

mfo commented 3 months ago

Tx pr tes retours ! juste un point a creuser que je ne suis pas sur de comprendre

top.

* Est ce que tu pourrais ajouter une option pour éditer ou supprimer l'email dans la boite d'alerte ?

hein ? mon attente sur la suggestion d'email : detecter les typo dans les X domaines. Soit on a un faux positif, soit c'est juste un faux. l'usager a le choix qu'entre ces deux cas non ?

* est ce que tu pourrais nous rajouter un peu de turbo pour qu'on ne remonte pas tout en haut de la page lorsqu'on soumet les emails ?

Pour le moment non, les composant react ne se remontent pas ac du turbo. J'avais remonté le point a tchak, en théorie la nouvelle implementation des combobox devraient le support.

* ca permettra de résoudre aussi des flash `success` et `alert` qui se superpose

pour le flash

Screencast.from.2024-06-24.10-14-35.webm

mfo commented 3 months ago

tx @LeSim – corrigé les differents point, ac en bonus le lien vers l'err : https://github.com/demarches-simplifiees/demarches-simplifiees.fr/pull/10524/commits/77bf93dfc2366fb45491cb623a20390c6203cf86