Change email catch for invite

rleger / covidrea

GNU Affero General Public License v3.0

3 stars 10 forks source link

Change email catch for invite #44

Open nyroDev opened 4 years ago

nyroDev commented 4 years ago

This should close #39 L'utilisation des noms extraits n'est pas fait et non sauvegardé pour le moment, mais il y a un @todo présent.

rleger commented 4 years ago

Merci @nyroDev
J'ai eu un souci lors d'un précédent merge ou la regex se comportait pas comme attendu et c'est un truc critique. @bdahon a fait un truc front end top entre temps. On garde la validation BE mais le FE reformatte avant. Je garde ton PR au chaud mais je merge pas tt de suite car il faut avoir des unit tests pour ce genre de fonction. Tu peux jeter un coup d'oeil a la solution FE de @bdahon

https://jsfiddle.net/pL1xwveu/2/

nyroDev commented 4 years ago

le code JS serait à exécuter non pas seulement au paste, mais aussi au keyup, change and input je dirai, histoire d'être tranquille dans tous les sens et quelque soit le moyen de renseigner la donnée.
Sinon ça fait le boulot aussi oui