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

Liste der Anlässe (3+3PT) #332

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

Liste der Anlässe (3+3PT) #332

fbufbi opened this issue Nov 30, 2021 · 41 comments
Labels
module-Anlassliste Feature für das Modul "Liste der Anlässe"
Milestone

Comments

@fbufbi
Copy link
Collaborator

fbufbi commented Nov 30, 2021

Einstiegspunkt für Details Anlass & Bewertung

  • Volltextsuchfeld nach den zwei Spalten Bezeichnung und Bewertung
    • sofortige Einschränkung der Tabelle
    • X zum einfachen Löschen des Suchbegriffs
  • Sortierung der Liste alphabetisch nach Bezeichnung.
  • Die Bezeichnung wird nicht ellipsiert, sondern umgebrochen (gewisse Ausbildungen haben sehr lange Fächerbezeichnungen).

image
image

Beispiele: GYM l1@test.ch (Klassenlehr- & Fachlehrperson), l3@test.ch (Fachlehrperson)

Endpunkte:

  • settings.eventlist.statusfilter: "14030;14025;14017;14020;10350;10335;10355;10315;10330;1032510320;10340;10345;10230;10225;10240;10260;10217;10235;10220;10226;10227;10250;10300"
  • GET /Courses/?expand=EvaluationStatusRef,AttendanceRef,Classes,FinalGrades&filter.statusId=;settings.eventlist.statusfilter, Request senden mit Header X-Role-Restriction: TeacherRole
  • Ist im token die roles: ClassTeacherRole vorhanden, Request senden mit Header X-Role-Restriction: ClassTeacherRole GET /StudyClasses/FormativeAssessments?filter.IsActive==true und /StudyClasses/?filter.IsActive==true. Alle Classes aus FormativeAssessments können bewertet werden. die anderen müssen nur angezeigt werden.

Felder:

  • Courses > Designation, Classes[x].Number (Komma getrennt), StudyClasses > Number -> link: settings.eventlist.eventdetail > Platzhalter {id} wird mit der id des Objekts ersetzt
  • DateFrom - DateTo
  • AttendanceRef.StudentCount

Status:

  • Bewertung
    • Events aus dem Endpunkt /StudyClasses/FormativeAssessments
    • link: settings.eventlist.evaluation > Platzhalter {id} wird mit der id des Objekts ersetzt
  • Bewertung bis EvaluationStatusRef.EvaluationUntil
    • EvaluationStatusRef.HasEvaluationStarted === true && EvaluationStatusRef.EvaluationUntil >= Date.Now && EvaluationStatusRef.HasTestGrading === false
    • link: settings.eventlist.evaluation > Platzhalter {id} wird mit der id des Objekts ersetzt
  • Zwischenbeurteilung
    • EvaluationStatusRef.HasEvaluationStarted === true && EvaluationStatusRef.EvaluationUntil === null && EvaluationStatusRef.HasTestGrading === false
    • link: settings.eventlist.evaluation > Platzhalter {id} wird mit der id des Objekts ersetzt
  • Test erfassen (Link auf Tests)
    • Label: locals.{lang}.events.state.events.state.add-tests > EvaluationStatusRef.HasTestGrading === true && EvaluationStatusRef.HasEvaluationStarted === false
    • Label: locals.{lang}.events.state.intermediate-rating > EvaluationStatusRef.HasTestGrading === true && EvaluationStatusRef.HasEvaluationStarted === true && EvaluationStatusRef.EvaluationUntil === null
    • Label: locals.{lang}.events.state.rating-until > EvaluationStatusRef.HasTestGrading === true && EvaluationStatusRef.HasEvaluationStarted === true && EvaluationStatusRef.EvaluationUntil >= Date.Now

Anlässe mit folgenden Bedingungen dürfen dargestellt werden:

  • EvaluationStatusRef.HasReviewOfEvaluationStarted === false || FinalGrades === null (expand=FinalGrades)
@fbufbi fbufbi added the module-Anlassliste Feature für das Modul "Liste der Anlässe" label Nov 30, 2021
@fbufbi fbufbi added this to the Prio 1 milestone Nov 30, 2021
@mburri
Copy link
Contributor

mburri commented Jan 24, 2022

Für den Request GET /Courses/?expand=EvaluationStatusRef,Classes haben wir bei der Evaluation/ Schätzung kein Feld StudentCount in der Response erhalten.

  • ist dieses Property optional? oder erst für bestimmte Status vorhanden?

Für den Request auf GET /StudyClasses/FormativeAssessments erhalten wir eine leere Response - hängt das mit unserem Testaccount l1@test.ch (GymHofwil) zusammen?

@schefbi
Copy link
Contributor

schefbi commented Jan 25, 2022

@mburri:

Für den Request GET /Courses/?expand=EvaluationStatusRef,Classes haben wir bei der Evaluation/ Schätzung kein Feld StudentCount in der Response erhalten.

Ist noch ein Bug wird noch ausgeliefert.

  • ist dieses Property optional? oder erst für bestimmte Status vorhanden?

Für den Request auf GET /StudyClasses/FormativeAssessments erhalten wir eine leere Response - hängt das mit unserem Testaccount l1@test.ch (GymHofwil) zusammen?

Ja der Testaccount ist zwar ClassTeacher hat aber keinen Klasse freigegeben. Ich habe nun eine freigegeben.

@lmathys lmathys changed the title Liste der Anlässe Liste der Anlässe (3+3PT) Jan 31, 2022
@caebr
Copy link
Collaborator

caebr commented Feb 16, 2022

@fbufbi Könntet ihr uns auch einen Testuser einrichten/angeben, der nicht ClassTeacher ist?

@caebr
Copy link
Collaborator

caebr commented Feb 16, 2022

@schefbi Das mit den Links habe ich noch nicht ganz verstanden. Diese werden im settings.js unter den angegebenen Variabeln definiert? Sind diese nicht teilweise dynamisch, also mit Id des entsprechenden Anlasses?
Könntest du da mal noch ein konkretes Beispiel machen?

@schefbi
Copy link
Contributor

schefbi commented Feb 17, 2022

@caebr Das ist ein gute Frage. Ich habe mir das vorgestellt, aber habe es nicht geschrieben. Ich habe mir ein Platzhalter vorgestellt {id} die beim rendern der HTML Seite ersetzt werden. Ich habe es in der Beschreibung ergänzt.

bsp. settings.eventlist.evaluation = '.../Evt_Pages/Brn_QualifikationDurchDozenten.aspx?IDAnlass={id}'

@fbufbi
Copy link
Collaborator Author

fbufbi commented Feb 17, 2022

@fbufbi Könntet ihr uns auch einen Testuser einrichten/angeben, der nicht ClassTeacher ist?

l3@test.ch

@caebr
Copy link
Collaborator

caebr commented Feb 17, 2022

@fbufbi Was bekommen Events für einen Status bzw. was wir im GUI angezeigt, wenn keine der oben genannten Bedingungen zutrifft?
Z.B. EvaluationStatusRef.HasEvaluationStarted === false && EvaluationStatusRef.HasTestGrading === false

Und zum Testen wäre es auch noch gäbig, wenn die Testdaten alle möglichen Status abbilden würden.

@fbufbi
Copy link
Collaborator Author

fbufbi commented Feb 17, 2022

@fbufbi Was bekommen Events für einen Status bzw. was wir im GUI angezeigt, wenn keine der oben genannten Bedingungen zutrifft? Z.B. EvaluationStatusRef.HasEvaluationStarted === false && EvaluationStatusRef.HasTestGrading === false

Kein Status, kein Text - die Zeile gibt es nicht, Siehe Fach Mathematik im Mockup zuunterst.

Und zum Testen wäre es auch noch gäbig, wenn die Testdaten alle möglichen Status abbilden würden.

Habe ich erfasst. Für l1 Klasse 22a mit Bewertung, Franz 22d Zwischenbewertung, Franz 24f Bewertung bis 28.07.2022, Franz 22a Tests erfassen und weitere. Für l3 alles ausser Klasse.

@schefbi
Copy link
Contributor

schefbi commented Mar 11, 2022

@caebr Es gibt eine Änderung auf Grund eines Fehlers der von SLH gefunden wurde und nun behoben ist.

  • settings.eventlist.statusfilter: "14030;14025;14017;14020;10350;10335;10355;10315;10330;1032510320;10340;10345;10230;10225;10240;10260;10217;10235;10220;10226;10227;10250;10300"
  • GET /Courses/?expand=EvaluationStatusRef,AttendanceRef,Classes&filter=;settings.eventlist.statusfilter, Request senden mit Header X-Role-Restriction: TeacherRole

Ich habe es auch in der Beschreibung ergänzt.

@caebr
Copy link
Collaborator

caebr commented Mar 14, 2022

@schefbi Ich setzte folgenden Request ab: https://evento-test.erz.be.ch/restApi/Courses/?expand=EvaluationStatusRef,AttendanceRef,Classes&filter=;14030;14025;14017;14020;10350;10335;10355;10315;10330;1032510320;10340;10345;10230;10225;10240;10260;10217;10235;10220;10226;10227;10250;10300

mit Header X-Role-Restriction: TeacherRole.

Das Liefert 116 Resultate zurück, darunter viele mit StatusId 10280. Der Filter scheint nicht zu funktionieren. Was mache ich falsch?

@caebr
Copy link
Collaborator

caebr commented Mar 14, 2022

@schefbi Ich setzte folgenden Request ab: https://evento-test.erz.be.ch/restApi/Courses/?expand=EvaluationStatusRef,AttendanceRef,Classes&filter=;14030;14025;14017;14020;10350;10335;10355;10315;10330;1032510320;10340;10345;10230;10225;10240;10260;10217;10235;10220;10226;10227;10250;10300

mit Header X-Role-Restriction: TeacherRole.

Das Liefert 116 Resultate zurück, darunter viele mit StatusId 10280. Der Filter scheint nicht zu funktionieren. Was mache ich falsch?

Ah - müsste es filter.StatusId= heissen?

@schefbi
Copy link
Contributor

schefbi commented Mar 14, 2022

@schefbi Ich setzte folgenden Request ab: https://evento-test.erz.be.ch/restApi/Courses/?expand=EvaluationStatusRef,AttendanceRef,Classes&filter=;14030;14025;14017;14020;10350;10335;10355;10315;10330;1032510320;10340;10345;10230;10225;10240;10260;10217;10235;10220;10226;10227;10250;10300
mit Header X-Role-Restriction: TeacherRole.
Das Liefert 116 Resultate zurück, darunter viele mit StatusId 10280. Der Filter scheint nicht zu funktionieren. Was mache ich falsch?

Ah - müsste es filter.StatusId= heissen?

Ja sorry er müsste filter.StatusId= heissen?

@schefbi
Copy link
Contributor

schefbi commented Mar 16, 2022

@caebr Wenn kein Status zutrifft wird dennoch der Link "Test erfassen" angezeigt. Kannst du das bitte noch ändern? Es darf kein Link in der Spalte Bewertung angezeigt werden.

@caebr
Copy link
Collaborator

caebr commented Mar 17, 2022

@schefbi Könntest du ein Beispiel der Daten bei denen es falsch angezeigt wird hier und in den Testdaten noch ergänzen?

Bzw. wäre das z.B. EvaluationStatusRef.HasTestGrading === false && EvaluationStatusRef.HasEvaluationStarted === false?

@schefbi
Copy link
Contributor

schefbi commented Mar 17, 2022

@caebr mit l3@test.ch ist es der CourseId 9467 der keine Bewertung zulässt. Wenn kein Status gemäss Beschreibung zutrifft gibt es kein Link.

@caebr
Copy link
Collaborator

caebr commented Mar 17, 2022

@caebr Wenn kein Status zutrifft wird dennoch der Link "Test erfassen" angezeigt. Kannst du das bitte noch ändern? Es darf kein Link in der Spalte Bewertung angezeigt werden.

Dieser Punkt ist nun korrigiert.

@caebr caebr unassigned mburri and caebr Mar 17, 2022
@fbufbi
Copy link
Collaborator Author

fbufbi commented Mar 17, 2022

@caebr Die Volltextsuche müsste alle Spalten umschliessen, nicht nur die Bezeichnung. Der Text im Suchfeld heisst dann einfach "Suchen..." statt "Bezeichnung suchen..." . Vgl. Mockup.

@fbufbi fbufbi assigned caebr and unassigned schefbi Mar 17, 2022
@schefbi
Copy link
Contributor

schefbi commented Mar 17, 2022

@caebr Ich musste noch eine Ergänzung machen. Beschreibung wie folgt angepasst:

  • Ist im token die roles: ClassTeacherRole vorhanden, Request senden mit Header X-Role-Restriction: ClassTeacherRole GET /StudyClasses/FormativeAssessments?filter.IsActive==true und /StudyClasses/?filter.IsActive==true. Alle Classes aus FormativeAssessments können bewertet werden. Die anderen müssen nur angezeigt werden.

Edit FBU:
Beispiel mit l1: Die Klasse 22a kann bewertet werden. Die Klasse 22x kann nicht bewertet werden, muss aber auch angezeigt werden. Die Klasse 21z ist nicht mehr aktiv und darf nicht erscheinen.

@schefbi
Copy link
Contributor

schefbi commented Mar 17, 2022

@caebr Bei der Anlassliste muss anstelle von Classes[x].Designation > Classes[x].Number angezeigt werden. Die Beschreibung wurde angepasst.

Beschreibung:

  • Designation, Classes[x].Number (Komma getrennt) -> link: settings.eventlist.eventdetail > Platzhalter {id} wird mit der id des Objekts ersetzt

@caebr
Copy link
Collaborator

caebr commented May 9, 2022

@schefbi Ich verstehe die zweite Bedingung nicht: FinalGrades: null > expand =FinalGrades. Und sind die beiden Bedingungn AND oder OR verknüpft?

caebr added a commit that referenced this issue May 9, 2022
@caebr
Copy link
Collaborator

caebr commented May 9, 2022

@schefbi Ich verstehe die zweite Bedingung nicht: FinalGrades: null > expand =FinalGrades. Und sind die beiden Bedingungn AND oder OR verknüpft?

Ah - es hat sich halb geklärt: Die zweite Bedingung ist FinalGrades === null und muss zusätzlich im Request geladen werden. Bezüglich AND/OR bin ich mir aber nicht sicher, nehme aber an AND?

@schefbi
Copy link
Contributor

schefbi commented May 10, 2022

@caebr Wie telefonisch besprochen die Anpassung in der Beschreibung

@schefbi
Copy link
Contributor

schefbi commented May 17, 2022

@caebr Beschreibung angepasst ich hoffe es ist verständlich, ansonsten bitte melden.
image

@caebr
Copy link
Collaborator

caebr commented May 17, 2022

@fbufbi Könntet ihr Anlässe für alle möglichen Fälle erfassen? Und vielleicht kurz beschreiben, bei welcher Anlass-Id, welcher Text/Link erwartet wird?

Danke

@fbufbi
Copy link
Collaborator Author

fbufbi commented May 18, 2022

@caebr

  • ID Anlass 9389: Bewertung bis, Link settings.eventlist.evaluation
  • ID Anlass 9248: Bewertung bis, Link auf Tests
  • ID Anlass 9250: Zwischenbewertung, Link settings.eventlist.evaluation
  • ID Anlass 9457: Zwischenbewertung, Link auf Tests
  • ID Anlass 9443: Link auf Tests (keine Bewertung)

mburri added a commit that referenced this issue May 18, 2022
build evaluation link independent from course state, refs #332
@caebr
Copy link
Collaborator

caebr commented May 19, 2022

@caebr Beschreibung angepasst ich hoffe es ist verständlich, ansonsten bitte melden. image

@fbufbi Das ist umgesetzt und auf master

@fbufbi
Copy link
Collaborator Author

fbufbi commented Jun 10, 2022

@schefbi Kannst du bitte testen:

  1. BS_FBU, kl@test.ch => Bewertung der Klasse => 0 Anmeldungen (damit kann die formative Beurteilung auf der Klasse nicht vorgenommen werden). Woran könnte das liegen?
    image
  2. GYM_TEST, l4: Mit a.ZwischenbewertungDefinitiv verhalten sich Anlässe mit und ohne Tests anders, wollen wir das so?
  • Mit Tests: Link "Tests erfassen"
  • Ohne Tests: kein Link

Alles Andere ist für mich Test OK

@fbufbi fbufbi reopened this Jun 10, 2022
@schefbi
Copy link
Contributor

schefbi commented Jun 14, 2022

@schefbi Kannst du bitte testen:

  1. BS_FBU, kl@test.ch => Bewertung der Klasse => 0 Anmeldungen (damit kann die formative Beurteilung auf der Klasse nicht vorgenommen werden). Woran könnte das liegen?
    image
  2. GYM_TEST, l4: Mit a.ZwischenbewertungDefinitiv verhalten sich Anlässe mit und ohne Tests anders, wollen wir das so?
  • Mit Tests: Link "Tests erfassen"
  • Ohne Tests: kein Link

Alles Andere ist für mich Test OK

  1. Bug Beta > mache ein Issue
  2. Ist von mir aus korrekt. Die LuL können weiter Tests erfassen, da dieser Status ja nur im Semester zugeteilt wird.

@fbufbi
Copy link
Collaborator Author

fbufbi commented Jun 17, 2022

@caebr Wir bräuchten auch hier noch einen zusätzlichen Eintrag für die französische Übersetzung "points" kleingeschrieben. Wahrscheinlich ist es zurzeit der gleiche Eintrag wie jener des Labels "Points" im Spaltenkopf der Tabelle:
image

@fbufbi fbufbi assigned caebr and unassigned schefbi Jun 17, 2022
@caebr
Copy link
Collaborator

caebr commented Jun 17, 2022

@caebr Wir bräuchten auch hier noch einen zusätzlichen Eintrag für die französische Übersetzung "points" kleingeschrieben. Wahrscheinlich ist es zurzeit der gleiche Eintrag wie jener des Labels "Points" im Spaltenkopf der Tabelle: image

Ist gemacht

@caebr caebr removed their assignment Jun 17, 2022
@fbufbi
Copy link
Collaborator Author

fbufbi commented Jun 24, 2022

Test OK, Anpassung der Links folgt in #427

@fbufbi fbufbi removed their assignment Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module-Anlassliste Feature für das Modul "Liste der Anlässe"
Projects
None yet
Development

No branches or pull requests

4 participants