update translations and fix release dialog

puzzle · Oct 25, 2024 · a5bf9ef · a5bf9ef
1 parent 0bf52b6
commit a5bf9ef
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 12 deletions.
diff --git a/frontend/cypress/e2e/objective.cy.ts b/frontend/cypress/e2e/objective.cy.ts
@@ -28,8 +28,9 @@ describe('OKR Objective e2e tests', () => {
           .get('.objective-menu-option')
           .contains('Objective veröffentlichen')
           .click();
+        cy.contains('Objective veröffentlichen');
+        cy.contains('Soll dieses Objective veröffentlicht werden?');
         cy.getByTestId('confirmYes').click();
-
         cy.getByTestId('objective')
           .filter(':contains(A objective in state draft)')
           .last()
@@ -151,6 +152,8 @@ describe('OKR Objective e2e tests', () => {
           .click()
           .wait(500)
           .tabForward();
+        cy.contains('Objective als Draft speichern');
+        cy.contains('Soll dieses Objective als Draft gespeichert werden?');
         cy.focused().click().wait(500);
 
         cy.getByTestId('objective')

diff --git a/frontend/src/app/components/objective/objective.component.ts b/frontend/src/app/components/objective/objective.component.ts
@@ -97,8 +97,8 @@ export class ObjectiveComponent implements OnInit {
           dialog: {
             dialog: ConfirmDialogComponent,
             data: {
-              title: 'Check-in im Draft-Status',
-              text: 'Dein Objective befindet sich noch im DRAFT Status. Möchtest du das Check-in trotzdem erfassen?',
+              title: 'Objective als Draft speichern',
+              text: 'Soll dieses Objective als Draft gespeichert werden?',
             },
           },
         },
@@ -107,18 +107,16 @@ export class ObjectiveComponent implements OnInit {
   }
 
   getDraftMenuActions() {
-    const releaseText = this.translate.instant('CONFIRMATION.RELEASE.TEXT');
-    const releaseBacklogText = 'releaseBacklog';
-    const dialogText = this.isBacklogQuarter ? releaseText : releaseBacklogText;
+    const action = this.isBacklogQuarter ? 'releaseBacklog' : 'release';
     let menuEntries = {
       displayName: 'Objective veröffentlichen',
-      action: this.isBacklogQuarter ? 'releaseBacklog' : 'release',
+      action: action,
       dialog: {
         dialog: this.isBacklogQuarter ? ObjectiveFormComponent : ConfirmDialogComponent,
         data: {
           title: this.translate.instant('CONFIRMATION.RELEASE.TITLE'),
-          text: dialogText,
-          action: dialogText,
+          text: this.translate.instant('CONFIRMATION.RELEASE.TEXT'),
+          action: action,
           objectiveId: this.isBacklogQuarter ? this.objective$.value.id : undefined,
         },
       },
@@ -158,7 +156,7 @@ export class ObjectiveComponent implements OnInit {
         data: {
           title: menuEntry.dialog.data.title,
           action: menuEntry.action,
-          text: menuEntry.action,
+          text: menuEntry.dialog.data.text,
           objective: menuEntry.dialog.data,
           objectiveTitle: menuEntry.dialog.data.objectiveTitle,
         },

diff --git a/frontend/src/assets/i18n/de.json b/frontend/src/assets/i18n/de.json
@@ -39,7 +39,7 @@
     },
     "TO_DRAFT": {
       "TITLE": "Objective als Draft speichern",
-      "TEXT": "Soll dieses ' + this.data.title + ' als Draft gespeichert werden?"
+      "TEXT": "Soll dieses Objective als Draft gespeichert werden?"
     },
     "DELETE": {
       "ACTION":{
@@ -55,7 +55,7 @@
         "TEXT": "Möchtest du dieses Objective wirklich löschen? Zugehörige Key Results werden dadurch ebenfalls gelöscht!"
       },
       "KEYRESULT":{
-        "TITLE": "Key Result löschen\n",
+        "TITLE": "Key Result löschen",
         "TEXT": "Möchtest du dieses Key Result wirklich löschen? Zugehörige Check-ins werden dadurch ebenfalls gelöscht!"
       }
     }