Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OKR Frontend - KeyResult (Resultat) löschen (mit Sicherheitsfrage) #333

Open
3 tasks
tuxmatta opened this issue Dec 10, 2022 · 1 comment
Open
3 tasks
Assignees

Comments

@tuxmatta
Copy link
Contributor

Als Kunde möchte ich ein KeyResult mit allen Messungen löschen, damit man fehlerhafte oder nicht mehr benötigte KeyResult entfernen kann.

Anforderungen

  • Ein KeyResult kann auf dem Dashboard im Menu der KeyResult-Zeile gelöscht werden.
  • Der Kunde muss vor dem Löschen eine Sicherheitsfrage (Dialog) bestätigen. Der Dialog sollte ähnlich dem Team löschen Dialog umgesetzt werden. Also auch eine Zusammenfassung anzeigen, was effektiv gelöscht wird:
    Team-löschen.png
  • Das Löschen kann einige Zeit dauern:
    Vorschlag:
  • Der Dialog wird offen gelassen und ein Angular Material Spinner wird im Dialog anstelle der Frage angezeigt. <mat-spinner></mat-spinner>
  • Nach dem Löschen wird der Spinner die Löschbestätigung oder die Fehlermeldung ersetzt.

Akzeptanzkriterien

  • Die Story ist wie beschrieben umgesetzt
  • Angular Komponente und das Template sind mit Jest getestet
  • Die benötigten Daten werden mittels Service + RxJS geladen.

Anpassungen an der Story

Anpassungen sind mit Jonas oder Paco vorab zu besprechen und an dieser Stelle einzutragen

@njaeggi njaeggi self-assigned this Dec 29, 2022
@njaeggi
Copy link

njaeggi commented Dec 30, 2022

Dieses Ticket ist nicht priorisiert, da nachträglich erst zu Sprint hinzugefügt wurde.

Löschen Button funktioniert, Seite wird neu geladen und auch eine erster Testversuch des Dialogs wurde implementiert.
https://github.com/puzzle/okr/tree/feature/354/keyresult-delete

Es fehlt:

  • Tests
  • Verbesserung des Dialogs
  • evt. bessere Lösung für das Neuladen
  • Ladeanzeige nach dem löschen

Falls möglich zuerst andere Tickets abschliessen, da ich ab Donnerstag wieder da bin udn hier weitermachen könnte.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants