calvinmclean/automated-garden

ActivePeriod should be considered when getting next water schedule

Closed this issue · 0 comments

Description

When getting the next WaterSchedule, the next run time is used, but this time might be outside of an ActivePeriod so I should change IsActive to accept time argument in case this next run time is in an inactive period.

It would also be good to change the response about next water time and duration to be its own struct because I can include more information about which WaterSchedule is being used or if there is no active WaterSchedule for the next watering time.