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

Edit booking period #3525

Merged
merged 1 commit into from Nov 14, 2023
Merged

Edit booking period #3525

merged 1 commit into from Nov 14, 2023

Conversation

Arashfa0301
Copy link
Contributor

Edit booking period

Copy link

codecov bot commented Nov 14, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (48d263b) 88.21% compared to head (b259fe4) 88.21%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3525   +/-   ##
=======================================
  Coverage   88.21%   88.21%           
=======================================
  Files         667      667           
  Lines       21089    21089           
=======================================
  Hits        18604    18604           
  Misses       2485     2485           

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

@Arashfa0301 Arashfa0301 merged commit 6c464ae into master Nov 14, 2023
3 checks passed
@Arashfa0301 Arashfa0301 deleted the edit-booking-period branch November 14, 2023 19:47
@@ -22,7 +22,7 @@
PENALTY_IGNORE_SUMMER = ((6, 1), (8, 15))
PENALTY_IGNORE_WINTER = ((11, 18), (1, 10))

BEDKOM_BOOKING_PERIOD = ((10, 2), (10, 19))
BEDKOM_BOOKING_PERIOD = ((1, 1), (12, 30))
Copy link
Member

Choose a reason for hiding this comment

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

Desember has 31 days, not 30. I assume nobody sends company interests on new years eve, but it's very random to have auto reply on every day of the year except for one...

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.

None yet

3 participants