Tests: Mittelwert setzen (2PT)

fbufbi commented 2 years ago

Status ermitteln für Mittelwert setzen und Link Bewertungsübersicht : isFinalGradeState: EvaluationStatusRef.HasEvaluationStarted === true && (EvaluationStatusRef.EvaluationUntil >= Date.Now || EvaluationStatusRef.EvaluationUntil === null)

Button «Mittelwert setzen» sichtbar wenn isFinalGradeState === true > action PUT /Courses/SetAverageTestResult body { "CourseIds": [{ids}] } Response: Neu gesetzte Gradings[x].GradeId im Frontend aktualisieren.
Footer: Link «Weiter zur Bewertung» sichtbar wennisFinalGradeState === true > Link: settings.eventlist.Evaluation;
Finale Note überschreiben wennisFinalGradeState === true && Gradings[x].CanGrade === true mittels PUT Gradings.HRef Body: {"GradeId":11,"GradeValue":null}

Gründe für "Umweg" über Bewertungsübersicht:

Teilnotenskala in Evento hinterlegen, aber Lehrkraft kann dann selber entscheiden, ob sie die Prüfungen überspringen und nur die Semesternoten auf der Seite «Bewertung» eingeben möchte

Eingabe, Kontrolle und/oder Korrektur von Absenzen (mit und ohne Absenzenverwaltung) auf der Seite «Bewertung»

Eingabe der formativen Beurteilung Ende des ersten Semesters der Mittelschulen auf der Seite «Bewertung»

to do FBI: Benachrichtigung an Lehrpersonen schicken, wenn Modulanlass 3 Tage vor Notenschluss noch auf «a.Beurteilung bis» steht

caebr commented 2 years ago

@schefbi Könntest du uns hier auch noch den Endpunkt zum Speichern der Note angeben bzw. erklären, was dieser macht.

schefbi commented 2 years ago

@schefbi Könntest du uns hier auch noch den Endpunkt zum Speichern der Note angeben bzw. erklären, was dieser macht.

@caebr Erledigt

mburri commented 2 years ago

Der Payload fürs Überschreiben des Mittelwertes der SuS ist {"GradeId": 2315} und nicht {"IdGrade": 2315}

GradeValue muss offenbar nicht mitgegeben werden.

mburri commented 2 years ago

@schefbi Uns ist unklar, wie das mit den Status des Kurses und der Sichtbarkeit des Buttons "Mittelwert setzen" gemeint ist - offenbar soll der Button nicht sichtbar sein, wenn der Kurs im Status "Tests erfassen" ist (so hat es Marc heute im Call verstanden) - aber das ist ja der einzige Status für den man in der Übersicht der Kurse auf "Tests erfassen" klickt und dann auch bei uns in der Applikation bleibt - sonst sind es ja immer externe Links.

Ist die Idee, dass man dann von den externen Systemen wieder zu uns springt um die Noten zu erfassen?

Mir ist auch aufgefallen, dass ich für den Kurs 9365 die Note eines SuS überschreiben kann, für den Kurs 9248 aber nicht, obwohl beide Kurse im Status "Tests erfassen" sind... interessanterweise ist für den Kurs 9248 für alle Gradings das Property CanGrade auf false gesetzt, für Kurs 9365 aber auf true - was bedeuted dieses Property und ist das für uns relevant?

schefbi commented 2 years ago

@mburri Sorry, das war mein Überlegungsfehler. Ich habe die Beschreibung angepasst. Der Status ob ein Finale Bewertung möglich ist muss sehr wohl anders ablaufen.

Das kann CanGrade Property gab es in der initial Version noch nicht. Ich habe es jetzt für das ermitteln der Finalen Noteneingabe mit einbezogen. Wir können das gerne am Daily nochmals anschauen.

mfehlmann commented 2 years ago

@schefbi Fehlt für den Status isFinalGradeState eventuell die Bedigung EvaluationStatusRef.HasTestGrading === True? Anstonsten wäre der Status gleich wie "Bewertung bis" || "Zwischenbeurteilung".

schefbi commented 2 years ago

@schefbi Fehlt für den Status isFinalGradeState eventuell die Bedigung EvaluationStatusRef.HasTestGrading === True? Anstonsten wäre der Status gleich wie "Bewertung bis" || "Zwischenbeurteilung".

@mfehlmann Jein, wenn Ihr es so baut das es nur aus der Sicht für Test funktionieren soll, dann Ja. Wenn wir aber diesen Status aus dem gesamten Kontext anschaut. Wie er jetzt definiert ist, kann die Finale Note gesetzt überschrieben oder gelöscht werden.

mfehlmann commented 2 years ago

Die Änderungen sind auf dem Master. Die Finale Note wird, wie gesagt, noch nicht aktualisiert - dafür haben wir aber eine Story

fbufbi commented 2 years ago

@mfehlmann Der Link "Weiter zur Bewertungsübersicht" wird angezeigt, wenn isFinalGradeState === true ist. Zurzeit wird der Link aber nur angezeigt, wenn zusätzlich mindestens 1 Test erfasst ist, das ist falsch. Beispiel Beurteilung ohne Tests: l1@test.ch, Fach Französisch, 24f (ID Anlass 9365).

mfehlmann commented 2 years ago

@fbufbi Soll der Link, wenn es keine Tests hat weiter oben stehen, oder nach wie vor untern rechts?

338_bewertungsübersicht_link

Bzw. braucht es den Text "Keine Tests vorhanden" dann noch?

fbufbi commented 2 years ago

@mfehlmann Wie wäre es so:

Der Footer (Linie + Link) rutscht nach oben, damit er schnell sichtbar ist (v.a. wenn ich für den Link nach unten scrollen müsste, wäre es gar nicht gut).
Der Text "Keine Tests vorhanden" kann stehen bleiben, sonst müssen wir es fast vom Link abhängig machen, was ich nicht gut finde. Es ist ja nicht falsch.

mfehlmann commented 2 years ago

@fbufbi Kannst du uns einen Kurs mit Tests einrichten, bei dem der Link kommen soll, damit wir das noch testen können.

fbufbi commented 2 years ago

@mfehlmann

ID Anlass 9248: Französisch, 22a => mit Tests, Bewertung bis
ID Anlass 9365: Französisch, 24f => ohne Tests, Bewertung bis (siehe https://github.com/bkd-mba-fbi/webapp-schulverwaltung/issues/338#issuecomment-1128479920)

fbufbi commented 2 years ago

@schefbi Beim Mittelwert setzen wird der oberste Textwert genommen, wenn jemand keine Tests eingetragen hat. Ist das korrekt? (Sollte zwar nicht vorkommen, aber trotzdem) l2 Fach Deutsch-S2, 22a

schefbi commented 2 years ago

@schefbi Beim Mittelwert setzen wird der oberste Textwert genommen, wenn jemand keine Tests eingetragen hat. Ist das korrekt? (Sollte zwar nicht vorkommen, aber trotzdem) l2 Fach Deutsch-S2, 22a

Ich finde das ein Fehler. Wenn der Durchschnitt nicht gerechnet werden kann aufgrund fehlender Test. Soll auch nichts eingetragen werden.

fbufbi commented 2 years ago

Test OK, Bug bei SLH in Arbeit

bkd-mba-fbi / webapp-schulverwaltung

Tests: Mittelwert setzen (2PT) #338