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

Tests: Mittelwert setzen (2PT) #338

Open
fbufbi opened this issue Nov 30, 2021 · 16 comments
Open

Tests: Mittelwert setzen (2PT) #338

fbufbi opened this issue Nov 30, 2021 · 16 comments
Assignees
Labels
module-Tests Feature für das Modul "Tests"
Milestone

Comments

@fbufbi
Copy link
Collaborator

fbufbi commented Nov 30, 2021

image

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 wenn isFinalGradeState === true > Link: settings.eventlist.Evaluation;
  • Finale Note überschreiben wenn isFinalGradeState === 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
@fbufbi fbufbi added the module-Tests Feature für das Modul "Tests" label Nov 30, 2021
@fbufbi fbufbi added this to the Prio 1 milestone Nov 30, 2021
@caebr
Copy link
Collaborator

caebr commented Jan 24, 2022

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

@schefbi
Copy link
Contributor

schefbi commented Jan 25, 2022

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

@caebr Erledigt

@lmathys lmathys changed the title Tests: Mittelwert setzen Tests: Mittelwert setzen (2PT) Jan 31, 2022
@mfehlmann mfehlmann self-assigned this Mar 31, 2022
@mburri
Copy link
Contributor

mburri commented Apr 27, 2022

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

GradeValue muss offenbar nicht mitgegeben werden.

@mburri
Copy link
Contributor

mburri commented Apr 27, 2022

@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
Copy link
Contributor

schefbi commented Apr 28, 2022

@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
Copy link
Contributor

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

@schefbi
Copy link
Contributor

schefbi commented Apr 28, 2022

@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
Copy link
Contributor

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

@fbufbi
Copy link
Collaborator Author

fbufbi commented May 17, 2022

@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
Copy link
Contributor

@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
Copy link
Collaborator Author

fbufbi commented May 17, 2022

@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.
    image

@mfehlmann
Copy link
Contributor

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

@fbufbi
Copy link
Collaborator Author

fbufbi commented May 17, 2022

@mfehlmann

@fbufbi
Copy link
Collaborator Author

fbufbi commented Jun 10, 2022

@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
Copy link
Contributor

schefbi commented Jun 14, 2022

@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
Copy link
Collaborator Author

fbufbi commented Jun 24, 2022

Test OK, Bug bei SLH in Arbeit

@fbufbi fbufbi assigned fbufbi and unassigned mfehlmann Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module-Tests Feature für das Modul "Tests"
Projects
None yet
Development

No branches or pull requests

5 participants