-
Notifications
You must be signed in to change notification settings - Fork 0
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
Comments
@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? |
@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). |
@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)? |
@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. |
@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. |
@hupf Aktualisieren, wenn Modul neu geladen oder ein anderer Tag gewählt wird. Plus Polling, deren Zeit in settings.js definiert werden kann. |
@fbufbi Hier 2 Varianten zum Review. |
@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). |
Danke @pmurkowsky @fbufbi – ich habe den Issue-Beschrieb entsprechend angepasst. |
@hupf Wenn in Vorlektion abwesend wird der Hinweis auf "Offene Absenzen" nicht angezeigt. Request: body:
Bei jeder Änderung des Status wird für jeden Student das Foto neu geladen. |
@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: Wann tritt dies auf? Gibt es Aktionen, die vorrangig ausgeführt werden müssen? |
@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. Wenn ich bei der ersten Person Abwesend wieder auf anwesend stelle erhalte ich folgendes Bild. |
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 |
Test OK |
Mockup:
The text was updated successfully, but these errors were encountered: