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 - Measure (Messung) - Resultat Beschreibung Diagramm #318

Open
3 tasks done
tuxmatta opened this issue Nov 28, 2022 · 5 comments
Open
3 tasks done

OKR Frontend - Measure (Messung) - Resultat Beschreibung Diagramm #318

tuxmatta opened this issue Nov 28, 2022 · 5 comments
Assignees

Comments

@tuxmatta
Copy link
Contributor

tuxmatta commented Nov 28, 2022

Als Kunde möchte ich bei den Measure (Messung) Resultat Beschreibung ein Diagramm mit der Entwicklung der Messung sehen, damit ich der Verlauf der Messung kontrollieren kann.

Screenshot from 2022-11-28 09-36-47.png

Beispiel-Diagramm

  • Die Ausgabe entspricht dem ScreenShot
  • Angular Komponente und das Template sind mit Jest getestet
  • Die benötigten Daten werden mittels Service + RxJS geladen.
@lkleisa lkleisa self-assigned this Dec 29, 2022
@lkleisa
Copy link

lkleisa commented Dec 30, 2022

Ich habe das Diagramm fertiggestellt. Es funktioniert jetzt, dass die einzelnen Messungen im Diagramm dargestellt werden und dass es sich an den Daten angepasst. Es gibt noch ein Problem mit der Darstellung, dass das Diagramm jetzt auf der rechten Seite der KeyResult Overview ist, jedoch noch viel zu klein. Dies habe ich noch nicht hinbekommen. Tests sind soweit für die Holung der Daten aus dem Service da, evt. gibt es noch eine Möglichkeit, das Diagramm selbst zu testten, dies hat es mir jedoch nicht mehr gerreicht anzuschauen.

@lkleisa
Copy link

lkleisa commented Dec 30, 2022

  • Tests überprüfen und evt. optimieren
  • Darstellung optimieren
  • Vergrösserung der Seite anschauen und schauen, ob sich das Diagramm so verhaltet, wie es soll

@Vakmeth Vakmeth self-assigned this Jan 3, 2023
@Vakmeth
Copy link

Vakmeth commented Jan 3, 2023

Vorgehen:

  • Auf Branch auschecken
  • Übersicht verschaffen
  • Checkliste von Lias (siehe oben), abarbeiten

@Vakmeth
Copy link

Vakmeth commented Jan 3, 2023

Was jetzt noch geändert wurde:

  • Das Diagramm wurde nun grösser gemacht und ist jetzt auch responsive

  • Um das Diagramm responisve zu machen, musste an der keyresult overview die Struktur leicht geändert werden: Wir arbeiten nun mit dem Bootstrap Grid, beim Handy wird das Diagramm bspw. nun nicht mehr rechts, sondern unterhalb den Informationen angezeigt

  • Zusätzlich habe ich noch einen Test hinzugefügt, welcher testet ob die DiagrammObjects korrekt erstellt wurden, dazu musste ich den Code aus der genereateDiagram Methode extrahieren und daraus eine eigene Methode machen. (So haben wir das ganze etwas enkapselt)

@tuxmatta tuxmatta added this to the BBT-Sprint-23-1 milestone Jan 4, 2023
@Vakmeth
Copy link

Vakmeth commented Jan 6, 2023

Geändert:

  • Das Goal wird nun auch an den diagram component gegeben, damit wir Zugriff auf den BasicValue und TargetValue haben
  • Diese Values werden benutzt um das provisorische min und max des Diagrams zu bestimmen
  • Entstehen Werte über dem Basicvalue und Targetvalue, so werden diese automatisch als min oder max gesetzt, da wir nun im setup des Diagrams suggestedMin und suggestedMax anstatt min und max verwenden.
  • Ebenfalls werden die Punkte so nicht mehr abgschnitten.

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

3 participants