#790 members einladen #793
Conversation
+ check if email not exists
…to feature/790_members_einladen # Conflicts: # frontend/src/app/team-management/invite-user-dialog/invite-user-dialog.component.spec.ts # frontend/src/app/team-management/invite-user-dialog/invite-user-dialog.component.ts # frontend/src/app/team-management/new-user/new-user.component.ts
…to feature/790_members_einladen
backend/src/main/java/ch/puzzle/okr/controller/UserController.java
Outdated
Show resolved
Hide resolved
| @if (index === 0) { | ||
| <label [for]="'firstName-col_' + index">Vorname</label> | ||
| } | ||
| <input |
There was a problem hiding this comment.
Die Inputs sind irgendwie nicht "standard":
Sie sind ecking und nicht so hoch wie gewöhnlich. Irgendwie scheint das Material Theme nicht angewendet werden.
There was a problem hiding this comment.
Das Padding hat noch gefehlt. Bezüglich den Ecken: ich habe dies den KeyResults angeglichen. Die Inputfelder sind schon etwas unterschiedlich.
frontend/src/app/team-management/new-user/new-user.component.html
Outdated
Show resolved
Hide resolved
frontend/src/app/team-management/invite-user-dialog/invite-user-dialog.component.html
Show resolved
Hide resolved
| @@ -141,6 +141,48 @@ describe('Team management tests', () => { | |||
| }); | |||
| }); | |||
|
|
|||
| describe('invite members', () => { | |||
There was a problem hiding this comment.
Would it be possible to add a test for the new user to log-in after registering, to verify the first or last name got updated and the assignment via email works? (Debatable, might be much effort to do)
| </div> | ||
| <div class="lastname equal-sized-col"> | ||
| <div class="input-style equal-sized-col"> | ||
| @if (index === 0) { |
There was a problem hiding this comment.
Ich finde es ein bisschen speziell einen index mitzugeben. Ist die Label-click logik für das fokusieren des inputs notwendig? Sonst könnte man wahrscheinlich auch ohne index auskommen.
Und: Wenn es 2 Zeilen sind und ich auf das Label klicke, dann springt es in das input der ersten Zeile.
(Könnte man aber evtl. auch so lassen)
There was a problem hiding this comment.
Wenn du das nicht über Labels machst, müsstest du die Titel wohl im äusseren invide-user-dialog.component.html hinzufügen. Dann hätte man wiederum das Problem, dass das Styling vielleicht nicht stimmt und die Spaltenbreite irgendwie kopiert werden müsste. Oder was wäre dann die Idee?
frontend/src/app/team-management/new-user/unique-mail.directive.ts
Outdated
Show resolved
Hide resolved
+ add emails from current form to unique-mail validator
…to feature/790_members_einladen
unique-mail validator - filter empty strings - update added mails every time form changes added general email validation
…to feature/790_members_einladen
…790_members_einladen
No description provided.