Merge pull request 'employee-detail-english' (#83) from employee-detail-english into main

Reviewed-on: #83
Reviewed-by: Jan Gleytenhoover <jan@kjan.email>

src/app/components/employee-detail/employee-detail.component.html

@@ -1,16 +1,16 @@
 <div class="container">
   <div class="d-flex flex-col" style="height: 100%;">
     <div class="p-3" style="width: 100%;">
-      <button class="btn btn-primary" (click)="goToEmployeePage()">Zurück</button>
+      <button class="btn btn-primary" (click)="goToEmployeePage()">Back</button>
       <div class="row align-items-start pt-3">
         <div class="col">
           <h1>{{ employee.firstName }} {{ employee.lastName }} </h1>
-          <p><strong>Straße: </strong>{{ employee.street }}</p>
-          <p><strong>Postleitzahl: </strong>{{ employee.postcode }}</p>
-          <p><strong>Stadt: </strong>{{ employee.city }}</p>
-          <p><strong>Telefonnummer: </strong>{{ employee.phone }}s</p>
-          <button class="btn btn-danger" (click)="deleteEmployee(currentId)">Löschen</button>
-          <button class="ms-3 btn btn-primary" (click)="editEmployee(currentId)">Bearbeiten</button>
+          <p><strong>Street: </strong>{{ employee.street }}</p>
+          <p><strong>Postal Code: </strong>{{ employee.postcode }}</p>
+          <p><strong>City: </strong>{{ employee.city }}</p>
+          <p><strong>Phone number: </strong>{{ employee.phone }}s</p>
+          <button class="btn btn-danger" (click)="deleteEmployee(currentId)">Delete</button>
+          <button class="ms-3 btn btn-primary" (click)="editEmployee(currentId)">Edit</button>
 
         </div>
         <div class="col">

src/app/components/mitarbeiter-form/mitarbeiter-form.component.ts

@@ -38,7 +38,7 @@ export class MitarbeiterFormComponent {
       postcode: new FormControl(this.mitarbeiter.postcode, [Validators.required, Validators.minLength(5), Validators.maxLength(5)]),
       city: new FormControl(this.mitarbeiter.city, Validators.required),
       phone: new FormControl(this.mitarbeiter.phone, [Validators.required, Validators.pattern('^[- +()0-9]+$')]),
-      newSkill: new FormControl(null, Validators.required) // Added
+      newSkill: new FormControl(null)
     });
   }
 
@@ -82,6 +82,9 @@ export class MitarbeiterFormComponent {
 
   addSkill() {
     const id = Number(this.mitarbeiterForm.get("newSkill")?.value);
+    if (!id) {
+      return;
+    }
     this.allSkills.subscribe(skills => {
       const newSkill = skills.filter(skill => skill.id == id)[0];
       this.mitarbeiter.skillSet?.push(newSkill);

src/app/components/qualifikation-form/qualifikation-form.component.ts

@@ -55,6 +55,9 @@ export class QualifikationFormComponent {
 
   addEmployee() {
     const employeeId = Number(this.skillForm.get("newEmployee")?.value);
+    if (!employeeId) {
+      return;
+    }
     const employee = this.addableEmployees.find(emp => emp.id === employeeId);
 
     if (employee) {