[ADD] #10 allow resetting future leaves for employees,

[hbrunn]

No description provided.

[albig]

Soweit ich das jetzt nachvollziehen konnte ist noch offen:

• Bearbeiten von Abwesenheiten vom Typ "Krankheit"
• Eintragen von "Krankheit" in vergangene Urlaubs-Abwesenheiten

Wenn ich hier falsch liege, brauch ich ne Einführung.

[hbrunn]

@albig hier soeben den Konflikt aufgelöst

[hbrunn]

@albig was war nochmal das Problem mit dem Bearbeiten von Krankheitstagen? Funktioniert das nicht genauso wie die anderen Urlaube (zu draft, dann bearbeiten, neu anfragen)?

[albig]

Bei Kranktagen funktioniert nur die Eintragung und Zurücksetzen zukünftiger Termine. Beispielsweise geplante Krankenhausaufenthalte. Insofern funktioniert das genauso, wie bei Urlaub.

Was noch nicht funktioniert, ist das Überlappen von nachgetragenen Kranktagen auf vergangen Urlaub.

Und der "Ablehnen"-Button für Mitarbeitende sollte weg, da man sowieso nicht die nötigen Rechte hat. Geht das?

[hbrunn]

ah danke, an den Überlappungen arbeite ich gerade, der erste Punkt von oben hat mich verwirrt.

Den Ablehnen-Knopf schaue ich mir danach an, aber vermutlich nicht heute

[albig]

Vermutlich war ich da selbst verwirrt, als ich das getestet habe. Aber es macht schon Sinn, dass auch Kranktage in der Vergangenheit nicht bearbeitet werden dürfen.

[hbrunn]

Ich habe das Modul auf https://github.com/hbrunn/hr-holidays/tree/15.0-hr_holidays_overlap gepushed. Da ich bisher nur den happy flow fertig implementiert und getestet habe will ich das noch nicht zum Testen/Review anbieten, aber falls Du selbst damit experimentieren willst, hier schonmal der Link.

Was geht: Beim Krankheitstyp 'Allow overlap' setzen, dann kannst Du Krankheitstage "über" Urlaubstage legen, und die sollten auch korrekt vom Saldo des Urlaubs abgezogen werden.

Was ich mir noch anschauen muss (gerne Feedback ob das für Euch überhaupt nötig ist, bzw Deine Meinung):

1. Interaktion mit abteilungsweiten Abwesenheiten
2. Wie soll sich das verhalten wenn mehr als zwei Abwesenheiten gestacked werden?
3. Dürfen Abwesenheiten desselben Types überlappen?
4. Soll der time_type der involvierten Urlaubstypen eine Rolle spielen?

1. weiss ich noch nicht, 2) würde ich verbieten, und bei 3+4 denke ich nein fur Version 1.

[albig]

Nur zum Verständnis: Statt dieses PRs, nehme ich jetzt https://github.com/hbrunn/hr-holidays/tree/15.0-hr_holidays_overlap? Das hat im ersten Versuch geklappt.

2. würde auch nicht zulassen.
3. nein, eher nicht.
4. Mit time_type meinst Du "Tag" oder "Stunden"? Das verwirrt mich ohnehin :-(

[hbrunn]

nehm beides, die zwei sind unabhängig voneinander. Ich habe bloss hier keine dependency zugefügt weil ich dem Code noch nicht vertraue. Den Teil hier kannst Du unabhängig mergen wenn das okay ist.

4. das ist das Feld Art des Urlaub / Abwesenheit, wo man einstellen kann ob das eine nicht-Arbeits-Abwesenheit ist (Urlaub/krank) oder eine Abwesenheit fur die Arbeit (Fortbildung etc). Zur Zeit erlaube ich das nur auf echten Abwesenheiten

[albig]

Bei diesem PR kommt ein Menü-Punkt "All leaves" für Administratoren hinzu, der das selbe Ergebnis liefert wie "All Time Off" (Approvals -> Time Off). Ist das so beabsichtigt?

Da beim PR auch das Eintragen von überlappenden Kranktagen auf Urlaub in der Vergangenheit nicht funktioniert, merg ich den Stand jetzt nicht.

Bei hr_holidays_overlap funktioniert das Eintragen von Kranktagen während Urlaub in der Vergangenheit. Es wird sofort der verfügbare Urlaub in der Anzeige erhöht. Allerdings steht dieser Urlaub dann nicht für neue Anträge zur Verfügung: Die verbleibende Abwesenheitszeit reicht für diese Abwesenheitsart nicht aus.
Bitte überprüfen Sie auch die Abwesenheit, die zur Validierung aussteht.

Die Urlaubsgutschrift erfolgt sofort, auch wenn eine Genehmigung aussteht. Auch genehmigte Krankheit löst es nicht aus, dass man diese Tage erneut für Urlaubsanträge nutzen kann.

[albig]

Bitte wie besprochen hier den Code aufräumen. Z.B. das Feld hr_leave_type.allow_overlap ist ja dann mit hr_leave_type.can_overlap aus hr_holidays_overlap hinfällig.

Den Sinn des Menü "All Leaves" kann ich nicht mehr nachvollziehen:

1. Mitarbeiter brauchen es nicht. Soweit klar.
2. Personalverantwortliche können unter Genehmigungen -> Abwesenheiten die Abwesenheiten "Ihrer" Mitarbeiter sehen
3. unternehmensweite Personalverantwortliche Verdigado Rolle "Manager") erhalten sowieso das Recht "Abwesenheiten: Administrator" und können alle Urlaube sehen.

[hbrunn]

hier dann nur das Bearbeiten, und hr_holidays_overlap funktioniert jetzt auch so wie es soll

[codecov-commenter]

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (b25ba7a) 71.14% compared to head (205cbe3) 69.58%.
Report is 2 commits behind head on 15.0.

Files	Patch %	Lines
verdigado_attendance/models/hr_leave.py	46.66%	7 Missing and 1 partial ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             15.0      #38      +/-   ##
==========================================
- Coverage   71.14%   69.58%   -1.56%     
==========================================
  Files          10       11       +1     
  Lines         201      217      +16     
  Branches        5        8       +3     
==========================================
+ Hits          143      151       +8     
- Misses         57       64       +7     
- Partials        1        2       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.