fixes_ui #66

Merged
Seil0 merged 3 commits from fixes_ui into master 2020-06-13 11:03:21 +02:00
Owner

closes:
#29
#65
#51

closes: #29 #65 #51
localhorst added the
review
label 2020-06-12 23:50:28 +02:00
Seil0 was assigned by localhorst 2020-06-12 23:50:28 +02:00
Seil0 requested changes 2020-06-13 10:26:04 +02:00
@ -8,0 +35,4 @@
Optional<User> user = userRepository.findByEmail(username);
if (!user.isPresent()) {
request.setAttribute("error", "Email Adresse falsch.");
Collaborator

Wäre "Die Email Adresse ist falsch." nicht besser?

Generell würde ich für Email und Passwort die gleiche Fehlermeldung zurück geben ("Die Email Adresse oder das Passwort ist falsch.").

Wäre "Die Email Adresse ist falsch." nicht besser? Generell würde ich für Email und Passwort die gleiche Fehlermeldung zurück geben ("Die Email Adresse oder das Passwort ist falsch.").
@ -8,0 +41,4 @@
}
if (!user.get().validatePassword(password)) {
request.setAttribute("error", "Passwort falsch.");
Collaborator

Wäre “Das Passwort ist falsch.” nicht besser?

Wäre “Das Passwort ist falsch.” nicht besser?
@ -8,0 +47,4 @@
}
if (!user.get().isActive) {
request.setAttribute("error", "User ist deaktiviert.");
Collaborator

Wäre “Dieses Konto ist deaktiviert.” nicht besser?

Wäre “Dieses Konto ist deaktiviert.” nicht besser?
@ -45,0 +31,4 @@
HttpSession session) {
Optional<User> user = userRepository.findByEmail(username);
if (user.isPresent()) {
request.setAttribute("error", "Email Adresse existiert bereits!");
Collaborator

Auch hier "Die Email Adresse existiert bereits." ?

Auch hier "Die Email Adresse existiert bereits." ?
@ -49,2 +39,2 @@
return "register";
}
if (!password.equals(password2)) {
request.setAttribute("error", "Passwörter sind nicht gleich");
Collaborator

"Die Passwörter sind nicht gleich." ?

"Die Passwörter sind nicht gleich." ?
@ -67,0 +56,4 @@
</div>
<fieldset>
<input type="radio" id="type-priv" name="type" value="priv">
<label for="male">Ich bin Privatkunde</label><br>
Collaborator

for noch auf die richtige ID setzen.

Auch bei type-bus, ad-y, ad-n

`for` noch auf die richtige ID setzen. Auch bei `type-bus`, `ad-y`, `ad-n`
Seil0 approved these changes 2020-06-13 11:03:14 +02:00
Seil0 left a comment
Collaborator

lgtm

lgtm
Seil0 closed this pull request 2020-06-13 11:03:21 +02:00
Seil0 deleted branch fixes_ui 2020-06-13 11:03:25 +02:00
This repo is archived. You cannot comment on pull requests.
No description provided.