Skip to content

Bugfix/#79 prevent negative work times#82

Merged
Death111 merged 5 commits intodevelopfrom
bugfix/#79_preventNegativeWorkTimes
Apr 11, 2021
Merged

Bugfix/#79 prevent negative work times#82
Death111 merged 5 commits intodevelopfrom
bugfix/#79_preventNegativeWorkTimes

Conversation

@JSchraff
Copy link
Contributor

@JSchraff JSchraff commented Apr 8, 2021

No description provided.

before start.

It will display an error message and prevent the dialog from closing.
Updates on change of time information.
@JSchraff JSchraff changed the base branch from master to develop April 8, 2021 08:14
@JSchraff JSchraff requested a review from Death111 April 8, 2021 08:14
Copy link
Collaborator

@Death111 Death111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed the logic to disable the OK button. Other than that this is fine.

@Death111 Death111 merged commit b355bf0 into develop Apr 11, 2021
@Death111 Death111 deleted the bugfix/#79_preventNegativeWorkTimes branch April 11, 2021 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants