cal icon indicating copy to clipboard operation
cal copied to clipboard

Published 20 hours ago • verified publisher icon rickar

Added WorkDurationInRange for better estimation of working days

Open Trial97 opened this issue 2 years ago • 3 comments

Trial97 avatar Mar 29 '22 09:03 Trial97

Thanks! Why would you use this over WorkHoursInRange or WorkdaysInRange?

rickar avatar Apr 02 '22 17:04 rickar

This one gives the exact Duration between two dates, of course excluding the holidays.

Trial97 avatar Apr 04 '22 12:04 Trial97

Ok, reading this again it looks like it's a non-inclusive range? I think that's going to be confusing when compared to WorkdaysInRange. Is there a reason for the difference?

If possible, could you rename this to WorkdaysInRangeDuration and make it use inclusive dates? The implementation could be something like return c.WorkdaysInRange(start, end) * time.Duration(24 * time.Hour)

rickar avatar Apr 13 '22 11:04 rickar

guys, do you have any progress here? may I help you somehow with this pr?

toffguy77 avatar Nov 02 '22 16:11 toffguy77