fix(booker): Recalculate today target time each day
The previous script was calculating target booking time at start. So when it was 20:00, it was not working correctly after the first day.
This commit is contained in:
@@ -192,8 +192,6 @@ class Booker:
|
|||||||
|
|
||||||
# Parse TARGET_RESERVATION_TIME to get the target hour and minute
|
# Parse TARGET_RESERVATION_TIME to get the target hour and minute
|
||||||
target_hour, target_minute = map(int, TARGET_RESERVATION_TIME.split(":"))
|
target_hour, target_minute = map(int, TARGET_RESERVATION_TIME.split(":"))
|
||||||
target_time = datetime.now(tz).replace(hour=target_hour, minute=target_minute, second=0, microsecond=0)
|
|
||||||
booking_window_end = target_time + timedelta(minutes=BOOKING_WINDOW_END_DELTA_MINUTES)
|
|
||||||
|
|
||||||
# Initial login
|
# Initial login
|
||||||
if not self.auth_handler.login():
|
if not self.auth_handler.login():
|
||||||
@@ -206,15 +204,19 @@ class Booker:
|
|||||||
current_time: datetime = datetime.now(tz)
|
current_time: datetime = datetime.now(tz)
|
||||||
logging.info(f"Current time: {current_time}")
|
logging.info(f"Current time: {current_time}")
|
||||||
|
|
||||||
|
# Recalculate target time and end time for the current day
|
||||||
|
today_target_time = current_time.replace(hour=target_hour, minute=target_minute, second=0, microsecond=0)
|
||||||
|
booking_window_end = today_target_time + timedelta(minutes=BOOKING_WINDOW_END_DELTA_MINUTES)
|
||||||
|
|
||||||
# Only book sessions if current time is within the booking window
|
# Only book sessions if current time is within the booking window
|
||||||
if target_time <= current_time <= booking_window_end:
|
if today_target_time <= current_time <= booking_window_end:
|
||||||
# Run booking cycle to check for preferred sessions and book
|
# Run booking cycle to check for preferred sessions and book
|
||||||
await self.booker(current_time)
|
await self.booker(current_time)
|
||||||
# Wait for a short time before next check
|
# Wait for a short time before next check
|
||||||
time.sleep(60)
|
time.sleep(60)
|
||||||
else:
|
else:
|
||||||
# Display message when outside booking window
|
# Display message when outside booking window
|
||||||
# logging.info(f"Not booking now - current time {current_time} is outside the booking window ({target_time} to {booking_window_end})")
|
# logging.info(f"Not booking now - current time {current_time} is outside the booking window ({today_target_time} to {booking_window_end})")
|
||||||
# Check again in 5 minutes if outside booking window
|
# Check again in 5 minutes if outside booking window
|
||||||
time.sleep(300)
|
time.sleep(300)
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
|
|||||||
Reference in New Issue
Block a user