rework buttons in objective form that the e2e test goes through and c…

…hange logic for the delete load off actions
puzzle · Jan 30, 2025 · 7edc767 · 7edc767
1 parent a356177
commit 7edc767
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 15 deletions.
diff --git a/frontend/cypress/e2e/objective.cy.ts b/frontend/cypress/e2e/objective.cy.ts
@@ -249,7 +249,7 @@ describe('okr objective', () => {
       cy.contains('Move to another quarter on edit');
     });
 
-    it.skip('should have primary button on all objective dialogs', () => {
+    it('should have primary button on all objective dialogs', () => {
       overviewPage
         .addObjective()
         .fillObjectiveTitle('A objective for testing purposes')

diff --git a/frontend/src/app/components/action-plan/action-plan.component.html b/frontend/src/app/components/action-plan/action-plan.component.html
@@ -12,7 +12,7 @@
   >
     <div
       class="container-fluid"
-      *ngFor="let item of getFormControlArray().controls!; index as i; trackBy: identity"
+      *ngFor="let item of getFormControlArray().controls!; index as i;"
       [formGroupName]="i"
       cdkDrag
       [cdkDragDisabled]="!movable"

diff --git a/frontend/src/app/components/action-plan/action-plan.component.spec.ts b/frontend/src/app/components/action-plan/action-plan.component.spec.ts
@@ -91,7 +91,7 @@ describe('ActionPlanComponent', () => {
   });
 
 
-  it.skip('should add new action with empty text into array', () => {
+  it('should add new action with empty text into array', () => {
     component.addNewItem();
     expect(component.getFormControlArray())
       .toHaveLength(1);

diff --git a/frontend/src/app/components/action-plan/action-plan.component.ts b/frontend/src/app/components/action-plan/action-plan.component.ts
@@ -16,7 +16,7 @@ import {
   FormControl,
   FormGroup
 } from '@angular/forms';
-import { identity, Observable, ReplaySubject } from 'rxjs';
+import { Observable, ReplaySubject } from 'rxjs';
 
 export type FormControlsOf<T> = {
   [P in keyof T]: AbstractControl<T[P]>;
@@ -82,12 +82,6 @@ export class ActionPlanComponent implements AfterContentInit {
           if (result) {
             this.getFormControlArray()
               .removeAt(index);
-            const rawValue = this.getFormControlArray()
-              .getRawValue() as Item[];
-            this.getFormControlArray()
-              .clear();
-            this.cdRef.detectChanges();
-            rawValue.forEach((t) => this.addNewItem(t));
             this.cdRef.detectChanges();
             if (item.id && this.onDelete) {
               this.onDelete(item.id)
@@ -126,6 +120,4 @@ export class ActionPlanComponent implements AfterContentInit {
   ngAfterContentInit(): void {
     this.addItemSubject.subscribe((item) => this.addNewItem(item));
   }
-
-  protected readonly identity = identity;
 }
diff --git a/frontend/src/app/shared/dialog/objective-dialog/objective-form.component.html b/frontend/src/app/shared/dialog/objective-dialog/objective-form.component.html
@@ -105,11 +105,22 @@
       </button>
       <span [ngClass]="data.objective.objectiveId ? 'pe-2': 'pe-2 ps-2'">
         <button
-          mat-button
-          *ngIf="allowedToSaveBacklog()"
+          mat-stroked-button
+          *ngIf="allowedToSaveBacklog() && !data.objective.objectiveId"
+          [disabled]="objectiveForm.invalid || !allowedToSaveBacklog()"
+          color="primary"
+          type="submit"
+          (click)="onSubmit('ONGOING')"
+          [attr.data-testId]="'save'"
+        >
+          Speichern
+        </button>
+
+        <button
+          mat-flat-button
+          *ngIf="allowedToSaveBacklog() && data.objective.objectiveId"
           [disabled]="objectiveForm.invalid || !allowedToSaveBacklog()"
           color="primary"
-          [ngClass]="data.objective.objectiveId ? 'mat-mdc-unelevated-button': 'mat-mdc-outlined-button'"
           type="submit"
           (click)="onSubmit('ONGOING')"
           [attr.data-testId]="'save'"