Peer Reviewing: Deutlicherer Badge für erfolgreich gereviewte Aufgaben

changed milestone to %2023 März Sprint

assigned to @eduard.frankford

assigned to @Michael.Breu and unassigned @eduard.frankford

Ich erlaube mir mal, das zu mir zu holen :-)

Michael

created branch 375-peer-reviewing-deutlicherer-badge-fur-erfolgreich-gereviewte-aufgaben to address this issue

Passt gut danke! LG Eduard

assigned to @eduard.frankford and unassigned @Michael.Breu

Hallo Eduard,

Habe im zugehörigen Branch 375-peer-reviewing-deutlicherer-badge-fur-erfolgreich-gereviewte-aufgaben die Batchlogik umgesetzt.

Folgende Punkte habe ich angepasst:

• Der Menu "Peer Reviewing" erscheint nun im Hauptmenu und hat nun ggf. einen "Badge"
• Es gibt nun in ReviewService.java eine Methode getReviewStatusCountsByCurrentUser die (für den aktuellen Nutzer) die Reviews nach Status in der Datenbank darstellt. Allerdings habe ich die genauen Review-"Stati" nicht nachvollziehen können. Ich habe zwar
  • Review In Progress
  • Review improved
  • Review completed gefunden. Das scheint aber irgendwie nicht vollständig zu sein (außerdem ist die Groß/Kleinschreibung nicht konsistent). Besser wäre wohl hier ein java-enum einzusetzen, und sicherzustellen, dass alle Zustände in der Datenbank landen.
• Auf Client-Seite werden die Statistiken gecached und nur erneut vom Server abgerufen, wenn der Nutzer Aktionen macht, die den Status potentiell verändert: z.B. mit return this.http.post<ReviewRequest>(this.resourceUrl + '/create', reviewRequest).pipe(tap(statistics => this.resetStatistics()));

Ich kann die Anzeige gerne noch verbessern, brauche aber Rückmeldungen zu den ReviewStati, bzw. gerne auch zur Darstellung.

Ach ja: deutsch/englisch Übersetzung ist eher noch schwach.

Michael

Hallo Michael,

schaut schon sehr gut aus, wäre es vlt noch sinnvoll auch beim Review "Requester" diese Badges anzuzeigen? Vorallem, wenn der status -> improvements requested ist, damit der Nutzer bemerkt, dass er seine Aufgabe verbessern soll? :-)

LG

mentioned in issue #377 (closed)

mentioned in merge request !179 (merged)

Ja natürlich, aber ich habe noch nicht genau verstanden, wie ich diese Info aus der Datenbank hole.

Momentan lese ich die Statistiken über "SELECT rr.status as status, count(*) as statusCount from ReviewRating rr WHERE rr.user = :user group by status" Wo finde ich denn denn "Requester" in der Datenbank?

Es wäre nützlich, eine State-Chart der Zustandsübergänge dazu zu haben

Ich kann das gerne ergänzen (die Infrastruktur haben wir ja jetzt). Aber ich verstehe die Datenbank Struktur zu wenig.

Klassendiagram/Statechart/SequenceChart wäre hilfreich

Ich würde alle reviews laden wo userID == requestedBy und dann checken ob status == "Improvement Requested" darüber eine Summe machen und dann oben anzeigen?

Sonst wenn man jede Änderung abbilden will, dann bräuchten wir die Review Historie oder?

assigned to @Michael.Breu and unassigned @eduard.frankford

Noch offene Punkte

• Badge "Peer Reviewed" wird erst beim Öffnen der Aufgabe angezeigt
• Hilfeseiten: Bilder werden (nicht mehr) angezeigt
• Marker in Menuleiste für weitere Reviewzustände erweitern

mentioned in merge request !180 (merged)

Hallo Eduard,

Ich habe die Badges jetzt soweit kurzfristig möglich umgesetzt:

• Reviewer sehen in der Navigationsleiste, wenn Reviews offen sind.
• Nutzer sehen, wenn Reviews angefordert, bzw. für Improvements zurückgewiesen wurden.
• Zudem sehen Nutzer, die Zahl der Aufgaben mit Badges.

Die Umsetzung ist noch nicht optimal, da

1. Die der Zustand der angeforderten Reviews nicht einfach aus der Datenbank ausgelesen werden kann. Den Zustand "Review Requested" gibt es leider nicht. Auch kann ein Review in mehreren Zuständen gleichzeitig sein, wenn mehrere Reviewer zugeordnet wurden.
2. Die zugeordneten Badges werden momentan auch aus den Reviews rausgelesen. Wenn ein Review gelöscht wurde, dann finden wir momentan den zugeordneten Badge nicht mehr :-(
3. Wenn ein Nutzer mehrfach einen Review für dieselbe Aufgabe anfordert ist es noch schwieriger

Ich denke, mit #384 (closed) wird das alles deutlich besser und einfacher. Dann kann man vieles direkt über elastic search machen.

_Ich habe die Badges jetzt soweit kurzfristig möglich umgesetzt:

• Reviewer sehen in der Navigationsleiste, wenn Reviews offen sind.
• Nutzer sehen, wenn Reviews angefordert, bzw. für Improvements zurückgewiesen wurden.
• Zudem sehen Nutzer, die Zahl der Aufgaben mit Badges._

Schaut gut aus, bei mir wurden teilweise negative Stände für die Requested Badges angezeigt, das habe ich mit 1f0a7995 gefixed.

1.Die der Zustand der angeforderten Reviews nicht einfach aus der Datenbank ausgelesen werden kann. Den Zustand "Review Requested" gibt es leider nicht. Auch kann ein Review in mehreren Zuständen gleichzeitig sein, wenn mehrere Reviewer zugeordnet wurden.

Ja stimmt das macht das alles ein bisschen komplizierter.

2. Die zugeordneten Badges werden momentan auch aus den Reviews rausgelesen. Wenn ein Review gelöscht wurde, dann finden wir momentan den zugeordneten Badge nicht mehr :-(

Find ich eigentlich gar nicht so verkehrt, wenn das badge nicht mehr angezeigt wird, wenn das review gelöscht wurde oder?

3.Die der Zustand der angeforderten Reviews nicht einfach aus der Datenbank ausgelesen werden kann. Den Zustand "Review Requested" gibt es leider nicht. Auch kann ein Review in mehreren Zuständen gleichzeitig sein, wenn mehrere Reviewer zugeordnet wurden.

-> Das sollte eigentlich nicht möglich sein oder?

Ich finde es schaut mittlerweile alles sehr gut aus. Was sind die nächsten Schritte?

LG Eduard

assigned to @eduard.frankford and unassigned @Michael.Breu

Habs gerade in den #211 (closed) 211-Review-Branch gemerged

done

closed