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.

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

Bug: Estimated/Actual Hours on work weeks are maxed at 24 hours #96

Closed
HankC138 opened this issue Feb 27, 2024 · 2 comments · Fixed by #102
Closed

Bug: Estimated/Actual Hours on work weeks are maxed at 24 hours #96

HankC138 opened this issue Feb 27, 2024 · 2 comments · Fixed by #102
Assignees
Labels

Comments

@HankC138
Copy link
Collaborator

Estimated and Actual Hours on workWeek are limited to 24 hours max, this should be 40 for an entire week? Do we need to account for the potential of more than 40 weeks in the case of potential overtime?

Copy link
Contributor

I think that having the option to add more than the current limit makes sense, especially for overtime. OTOH this brings up a good point; do we want to warn for overtime somehow?

@fermion
Copy link
Collaborator

fermion commented Feb 27, 2024

@MyStarrySpace this is just my bad. When I was building this out initially I meant to put in a validation that the hours can't be more than there are hours in any one week. You can't possibly work 65,000 hours in a week, for example. I was going to bump this up 168 for the week, though that would be one hella week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants