Schenk-Smart-Solutions / web-components-toolbox-musigschuel

musigschuel
https://schenk-smart-solutions.github.io/web-components-toolbox-musigschuel/
Other
0 stars 0 forks source link

RegistrationForm: Registrierung mit e-mail ohne TLD möglich #10

Closed roli81 closed 7 months ago

roli81 commented 7 months ago

Es ist möglich sich mit ungültigen E-Mail Adressen wie music-guy@gmail zu registrieren. @Weedshaker Bitte Regex validierung anpassen, dass eine TLD eingegeben werden muss!

Image

Weedshaker commented 7 months ago

Gemäss Web Standart: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/email#value username@domain or username@domain.tld

roli81 commented 7 months ago

@Weedshaker der standard ist mir hier eigentlich egal. Es führte dazu das sich ein Benutzer zweimal registrieren konnte und die einfachste Art bist es jetzt abzufangen das keine emails ohne TLD möglich sind. Ich habs jetzt halt mal im backend gemacht aber ist halt so naja. Um die ganze Logik auf Standard umzubauen habe ich wieder zwei stunden die Registrierung ist der komplizierte Teil und sorry kein schwein hat eine email ohne tld.

Weedshaker commented 7 months ago

<input type="email" name="user_email" pattern="[a-z0-9._%+\-]+@[a-z0-9.\-]+\.[a-z]{2,}$"> then just put the pattern attribute with the right regex into the desired input field.

Weedshaker commented 7 months ago

I make you an example

Weedshaker commented 7 months ago

da war noch ein fehler im pattern> pattern="[a-z0-9._%+-]+@[a-z0-9.-]+.[a-z]{2,}$

Weedshaker commented 7 months ago

siehe hier: https://github.com/Schenk-Smart-Solutions/web-components-toolbox-musigschuel/commit/e01a88b1ee800dcbc765fef9dd9a78428f44537d#diff-223e409bef811d5cb17794b434a752f76c438e2602d158d896e59c02fa6cbb71R36 oder https://github.com/Schenk-Smart-Solutions/web-components-toolbox-musigschuel/blob/master/src/es/components/pages/FormApi.html#L36

Weedshaker commented 7 months ago

@roli81 , der Pattern wurde noch korrigiert. [a-z0-9._%+\-]+@[a-z0-9.\-]+.[a-z]{2,}$ der warf einen Error in der Dev Tools Konsole. Jetzt geht es... kann hier getestet werden: https://schenk-smart-solutions.github.io/web-components-toolbox-musigschuel/src/es/components/web-components-toolbox/docs/TemplateNoControllers.html?rootFolder=src&css=./src/css/variablesCustom.css&logo=./src/es/components/atoms/logo/default-/default-.html&nav=./src/es/components/molecules/navigation/default-/default-.html&footer=./src/es/components/organisms/footer/default-/default-.html&content=./src/es/components/pages/FormApi.html