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

Präsenzkontrolle: Hinweis zu offenen Absenzen (1PT) #149

Closed
fbufbi opened this issue Mar 4, 2020 · 15 comments
Closed

Präsenzkontrolle: Hinweis zu offenen Absenzen (1PT) #149

fbufbi opened this issue Mar 4, 2020 · 15 comments
Assignees
Labels
module-Präsenzkontrolle Feature fürs Modul "Präsenzkontrolle"

Comments

@fbufbi
Copy link
Collaborator

fbufbi commented Mar 4, 2020

  • Wenn für eine/n Schüler/in offene Absenzen vorhanden sind, soll unter dem Namen ein Hinweis "Offene Absenzen" angezeigt werden (rot wie Absenz)
  • Die Daten analog "Offene Absenzen" laden (sind zwei Requests – Fachlehrkraft sieht Markierung nur, wenn nicht die Klassenlehrkraft die Absenzen bestätigt). Aktualisieren, wenn Modul neu geladen oder ein anderer Tag gewählt wird. Plus Polling, dessen Zeit in settings.js definiert werden kann.
  • Klick auf Hinweis führt auch auf das Schüler-Profil.

Mockup:
image

@fbufbi fbufbi added the module-Präsenzkontrolle Feature fürs Modul "Präsenzkontrolle" label Mar 4, 2020
@fbufbi fbufbi added this to the 1_Präsenzkontrolle milestone Mar 4, 2020
@fbufbi fbufbi changed the title Präsenzkontrolle: Name gelb hinterlegt, wenn offene Absenzen vorhanden Präsenzkontrolle: Hinweis zu offenen Absenzen Mar 5, 2020
@hupf hupf added the konzeption Task für Konzeption (POs & UX) label Apr 20, 2020
@hupf
Copy link
Collaborator

hupf commented Apr 20, 2020

@fbufbi Werden für die Anzeige des Hinweises alle offene Absenzen berücksichtigt, also auch solche vom aktuellen Tag? D.h. wenn ich von grün auf rot wechsle, erscheint sofort auch der Hinweis?

@fbufbi
Copy link
Collaborator Author

fbufbi commented Apr 21, 2020

@hupf Ja, das ist die logische Konsequenz, denn ab diesem Zeitpunkt sind die Absenzen auch im Arbeitsgebiet "Offene Absenzen" sichtbar. Es sollen ja dieselben Requests verwendet werden (siehe Beschreibung; es sind zwei Requests – die Fachlehrkraft sieht die Markierung nur, wenn nicht die Klassenlehrkraft die Absenzen bestätigt).

@hupf
Copy link
Collaborator

hupf commented Apr 21, 2020

@fbufbi Ich denke es macht wenig Sinn bei jedem Statuswechsel beide Requests auszuführen, da wir in diesem Fall ja wissen wie sich die Offenen Absenzen verändern und lokal updaten können. Wir müssen uns aber überlegen, wann wir diese Offene Absenzen aktualisieren. Bei einem Wechsel der Lektion? Bei einem Wechsel des Tages? Polling (z.B. alle 5 Minuten aktualisieren)?

@fbufbi
Copy link
Collaborator Author

fbufbi commented Apr 21, 2020

@hupf Ach so. In der Beschreibung steht "einmalig laden, wenn das Modul besucht [geladen] wird". D.h. bei einem Wechsel der Lektion: nein, bei einem Wechsel des Tages: ja.

@hupf
Copy link
Collaborator

hupf commented Apr 21, 2020

@fbufbi Ist es in den meisten Fällen so, dass die Hinweise für selber erstellte Offene Absenzen sieht? Dann reicht einmaliges Laden völlig. Wenn man auch Offene Absenzen sieht die andere Lehrkräfte erstellt haben -> häufiger aktualisieren.

@fbufbi
Copy link
Collaborator Author

fbufbi commented Apr 21, 2020

@hupf Aktualisieren, wenn Modul neu geladen oder ein anderer Tag gewählt wird. Plus Polling, deren Zeit in settings.js definiert werden kann.
Bei Änderung wird also nichts sofort angezeigt (da sehr aufwändige Logik), erst mit dem Polling.

@lmathys lmathys changed the title Präsenzkontrolle: Hinweis zu offenen Absenzen Präsenzkontrolle: Hinweis zu offenen Absenzen (1PT) Apr 21, 2020
@pmurkowsky
Copy link

@fbufbi Hier 2 Varianten zum Review.

Offene-Absenzen-auf-Präsenzkontrolle

@fbufbi
Copy link
Collaborator Author

fbufbi commented Apr 24, 2020

@pmurkowsky Danke für die Vorschläge. Uns gefällt Variante 1 besser. Variante 2 hat was, aber der Badge ist zu dominant, das Auge wird immer wieder dorthin gezogen. Daher für uns Variante 1, da sie den Zweck erfüllt, ohne weitere Erklärung verständlich ist (Text) und das Auge nicht ablenkt. Farbe Rot ist guter Hinweis (da Absenz).

@pmurkowsky pmurkowsky removed the konzeption Task für Konzeption (POs & UX) label Apr 26, 2020
@hupf
Copy link
Collaborator

hupf commented Apr 27, 2020

Danke @pmurkowsky @fbufbi – ich habe den Issue-Beschrieb entsprechend angepasst.

@hupf hupf self-assigned this Apr 29, 2020
@hupf
Copy link
Collaborator

hupf commented Apr 30, 2020

Im Grid Layout habe ich es mit Lücke bei keinem Hinweis umgesetzt:

image


Über das Listenlayout haben wir nicht gesprochen, meiner Meinung nach sieht es gut aus analog Grid (Name/Hinweis werden vertikal zentriert und rücken etwas nach oben):
image


Auch bei schmaler Liste passt der Hinweis so ganz gut:

Was meint ihr, @fbufbi @schefbi

@hupf hupf assigned caebr and unassigned hupf Apr 30, 2020
@caebr caebr assigned hupf and unassigned caebr May 4, 2020
@hupf hupf assigned schefbi and fbufbi and unassigned hupf May 4, 2020
@schefbi
Copy link
Contributor

schefbi commented May 5, 2020

@hupf Wenn in Vorlektion abwesend wird der Hinweis auf "Offene Absenzen" nicht angezeigt.
image

Request: /LessonPresences/?filter.TypeRef==11&filter.ConfirmationStateId==219&filter.HasStudyCourseConfirmationCode==true

body:

[{
    "Id": "7395475_37002",
    "LessonRef": {
      "Id": 7395475,
      "HRef": null
    },
    "StudentRef": {
      "Id": 37002,
      "HRef": "/restApi/Students/37002"
    },
    "StudentFullName": "Student NOK",
    "EventRef": {
      "Id": 45014,
      "HRef": "/restApi/EducationalEvents/45014"
    },
    "EventDesignation": "Berufskenntnisse",
    "EventNumber": "1_2_ABKT_BSB_FAGE_R17_FAGE2019f",
    "EventTypeId": 3,
    "StudyClassRef": {
      "Id": 42801,
      "HRef": "/restApi/StudyClasses/42801"
    },
    "StudyClassDesignation": "Fachmann/frau Gesundheit EFZ 2019f",
    "StudyClassNumber": "FAGE2019f",
    "LessonDateTimeFrom": "2020-05-05T11:20:00",
    "LessonDateTimeTo": "2020-05-05T12:05:00",
    "IsReadOnly": false,
    "Date": null,
    "TypeRef": {
      "Id": 11,
      "HRef": "/restApi/PresenceTypes/11"
    },
    "Type": "Abwesend",
    "Comment": null,
    "ConfirmationStateId": 219,
    "ConfirmationState": null,
    "HasStudyCourseConfirmationCode": false,
    "WasAbsentInPrecedingLesson": null,
    "TeacherInformation": "test"
  },
  {
    "Id": "7395472_40595",
    "LessonRef": {
      "Id": 7395472,
      "HRef": null
    },
    "StudentRef": {
      "Id": 40595,
      "HRef": "/restApi/Students/40595"
    },
    "StudentFullName": "Student OK",
    "EventRef": {
      "Id": 45014,
      "HRef": "/restApi/EducationalEvents/45014"
    },
    "EventDesignation": "Berufskenntnisse",
    "EventNumber": "1_2_ABKT_BSB_FAGE_R17_FAGE2019f",
    "EventTypeId": 3,
    "StudyClassRef": {
      "Id": 42801,
      "HRef": "/restApi/StudyClasses/42801"
    },
    "StudyClassDesignation": "Fachmann/frau Gesundheit EFZ 2019f",
    "StudyClassNumber": "FAGE2019f",
    "LessonDateTimeFrom": "2020-05-05T08:35:00",
    "LessonDateTimeTo": "2020-05-05T09:20:00",
    "IsReadOnly": false,
    "Date": null,
    "TypeRef": {
      "Id": 11,
      "HRef": "/restApi/PresenceTypes/11"
    },
    "Type": "Abwesend",
    "Comment": null,
    "ConfirmationStateId": 219,
    "ConfirmationState": null,
    "HasStudyCourseConfirmationCode": false,
    "WasAbsentInPrecedingLesson": null,
    "TeacherInformation": "test"
  }]

Bei jeder Änderung des Status wird für jeden Student das Foto neu geladen.
image

@schefbi schefbi assigned hupf and unassigned schefbi and fbufbi May 6, 2020
@hupf
Copy link
Collaborator

hupf commented May 6, 2020

@schefbi Ich habe versucht das Verhalten mit dem Neuladen der Bilder zu reproduzieren, kriege es aber nicht hin. Bei mir wird jeweils nur der PUT Request gemacht, die Bilder werden nur initial oder bei einem Wechsel der Lektion neu geladen:

#160 (comment)
image

Wann tritt dies auf? Gibt es Aktionen, die vorrangig ausgeführt werden müssen?

@schefbi
Copy link
Contributor

schefbi commented May 6, 2020

@hupf Ich kann es auch nicht mehr nachstellen. Das die Fotos nach dem Update neu geladen werden. Der einzige unterschied ist VPN. Ich werde es nochmals mit VPN anschauen.

Zum Problem mit den Offenen Absenzen kann ich folgendes sagen.

2 Personen gleiche Lektion abwesend gestellt.
image

Nachfolge Lektion
image

Wenn ich bei der ersten Person Abwesend wieder auf anwesend stelle erhalte ich folgendes Bild.
image

@hupf
Copy link
Collaborator

hupf commented May 12, 2020

Uh ja, das war ein Bug von mir, sorry. Die Idee mit dem ersten Element in der Response war gut, @schefbi.

Habe in diesem Rahmen gleich alle arr.indexOf(x) !== -1 mit arr.includes(x) ersetzt (wird für IE11 polyfilled).

@hupf hupf assigned schefbi and unassigned hupf May 12, 2020
@fbufbi
Copy link
Collaborator Author

fbufbi commented May 13, 2020

Test OK
@schefbi Dispensation & "In Lektion abwesend" funktioniert

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module-Präsenzkontrolle Feature fürs Modul "Präsenzkontrolle"
Projects
None yet
Development

No branches or pull requests

5 participants