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

[ADD] allow deleting breaks #47

Merged
merged 1 commit into from
Oct 30, 2023
Merged

[ADD] allow deleting breaks #47

merged 1 commit into from
Oct 30, 2023

Conversation

hbrunn
Copy link
Contributor

@hbrunn hbrunn commented Oct 27, 2023

@albig
Copy link
Member

albig commented Oct 27, 2023

Danke @hbrunn. Mit der Änderungen in hr_attendance_break funktioniert es, dass überlappende Pausen nicht mehr möglich sind.

Zwei Dinge:

  1. Kannst Du noch sicherstellen, dass Endzeitpunkt > Startzeitpunkt ist? Beim Eintragen macht man da gern ein Fehler, weil das Formular mit Startzeitpunkt = Endzeitpunkt vorausgefüllt ist.
  2. Die Tests bei diesem PR schlagen fehl. Der Grund sagt mir leider nix. Hast du dafür eine Erklärung?

@albig albig mentioned this pull request Oct 30, 2023
Closed
@albig albig linked an issue Oct 30, 2023 that may be closed by this pull request
Löschen von Pausen für normale Mitarbeitende (Prio 1) #41
Closed
@albig albig merged commit feb7e24 into verdigado:15.0 Oct 30, 2023
1 of 4 checks passed
@hbrunn
Copy link
Contributor Author

hbrunn commented Nov 13, 2023

@albig CI habe ich in #53 repariert, und den check fur Beginn/Ende in OCA/hr-attendance@a52ffd9 zugefügt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Löschen von Pausen für normale Mitarbeitende (Prio 1)
2 participants
@hbrunn @albig