jank/employeeService
1
1
Fork 0
Code Issues 2 Pull requests 4 Projects 1 Releases Packages Wiki Activity Actions

Merge pull request 'Fix selecting nothing' (#81) from fix/fix-selecting-nothing-as-skill into main
All checks were successful
Playwright Tests / test (push) Successful in 1m55s
Details

Browse source 
Reviewed-on: #81
This commit is contained in:
Jan Gleytenhoover 2025-01-22 11:17:44 +00:00
commit 6efe952831
2 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/app/components/mitarbeiter-form/mitarbeiter-form.component.ts
View file

@ -82,6 +82,9 @@ export class MitarbeiterFormComponent {
addSkill() { addSkill() {
const id = Number(this.mitarbeiterForm.get("newSkill")?.value); const id = Number(this.mitarbeiterForm.get("newSkill")?.value);
if (!id) {
return;
}
this.allSkills.subscribe(skills => { this.allSkills.subscribe(skills => {
const newSkill = skills.filter(skill => skill.id == id)[0]; const newSkill = skills.filter(skill => skill.id == id)[0];
this.mitarbeiter.skillSet?.push(newSkill); this.mitarbeiter.skillSet?.push(newSkill);

3
src/app/components/qualifikation-form/qualifikation-form.component.ts
View file

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