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.

Sign up for GitHub

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

Jump to bottom

Absenzen bearbeiten: Grund als Vorauswahl / Grund übernehmen #211

Closed
fbufbi opened this issue Sep 25, 2020 · 3 comments
Closed

Absenzen bearbeiten: Grund als Vorauswahl / Grund übernehmen #211

fbufbi opened this issue Sep 25, 2020 · 3 comments
Assignees
@fbufbi
Labels
module-AbsenzenBearbeiten Feature fürs Modul "Absenzen bearbeiten"

Comments

@fbufbi
Copy link
Collaborator

fbufbi commented Sep 25, 2020 •
edited by hupf
Loading

  1. Ist ein Grund vorhanden bzw. ist er bei mehreren selektierten Lektionen gleich, soll er als Vorauswahl eingeblendet werden
  2. Sind in selektierten Lektionen ein oder mehrere unterschiedliche Gründe vorhanden, sollen sie mit "zu bestätigen" und "zu kontrollieren" übernommen werden (kein Grund setzen). Zurzeit wird PresencesTypes.Id = 11 (abwesend) gesetzt und damit der Grund gelöscht. Bei "unentschuldigt" soll das Verhalten gleich bleiben.

image

GYM, l1@test.ch, 02.11.2020 für Schülerin Fankhauser L.
@fbufbi fbufbi added the module-AbsenzenBearbeiten Feature fürs Modul "Absenzen bearbeiten" label Sep 25, 2020
@fbufbi fbufbi changed the title Absenzen bearbeiten: zu bestätigen - Grund übernehmen / zu kontrollieren ausblenden Absenzen bearbeiten: Grund als Vorauswahl / Grund übernehmen Oct 22, 2020
@hupf hupf self-assigned this Oct 27, 2020
@hupf
Copy link
Collaborator

hupf commented Oct 27, 2020

Problem:
Wenn wir beim Wechsel auf "zu bestätigen" oder "zu kontrollieren" den PresenceType nicht mehr setzen, wird der ConfirmationValue für Einträge die aktuell eine Dispensation, ein Halbtag oder ein Vorfall sind vom Backend nicht geändert (denn diese Kombination ist fachlich nicht möglich). In diesem Fall müsste explizit für diese Einträge der PresenceType auf 11 gesetzt werden (wie aktuell implementiert). D.h. wir müssten je nach Auswahl zwei Requests machen und die Einträge unterschiedlich behandeln.

Entscheid mit @fbufbi & @schefbi:
Falls mind. eine Dispensation, ein Halbtag oder ein Vorfall in der Auswahl vorhanden ist, werden vorläufig die Radios "zu bestätigen", "zu kontrollieren" disabled. Der User kann dann immer noch auf "Anwesend wechseln" und dann in einem zweiten Schritt auf "zu bestätigen" oder "zu kontrollieren".

hupf added a commit that referenced this issue Oct 28, 2020
@hupf
Preselect absence type and don't override existing absence types on e…
4bda2e2 
…dit #211
hupf added a commit that referenced this issue Oct 28, 2020
@hupf
Preselect absence type and don't override existing absence types on e…
3989d2d 
…dit #211
@hupf
Copy link
Collaborator

hupf commented Oct 28, 2020

Umgesetzt ist nun die Variante mit zwei Requests, falls nötig.

@hupf hupf assigned caebr and unassigned hupf Oct 28, 2020
hupf added a commit that referenced this issue Oct 28, 2020
@hupf
Preselect absence type and don't override existing absence types on e…
18b8c1e 
…dit #211
@hupf hupf assigned fbufbi and unassigned caebr Oct 28, 2020
@fbufbi
Copy link
Collaborator Author

fbufbi commented Oct 29, 2020

Test OK

@fbufbi fbufbi closed this as completed Nov 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fbufbi fbufbi

Labels
module-AbsenzenBearbeiten Feature fürs Modul "Absenzen bearbeiten"
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@hupf @caebr @fbufbi