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

State planning across timezone changes #129

Open
ondryaso opened this issue Oct 31, 2022 · 0 comments
Open

State planning across timezone changes #129

ondryaso opened this issue Oct 31, 2022 · 0 comments
Labels
bug Something isn't working comp-states Related to the state management functionality low-priority A low-priority issue

Comments

@ondryaso
Copy link
Member

When time(zone) changes while the service is running and StatePlannerBackgroundService is sleeping, having previously received a nextTransition state, the planner service fires events at wrong times – because it doesn't have any way of knowing that the amout of time to wait should be adjusted.

This should be partially solved by properly handling timezones (as previously mentioned in #55). Also, I think that an additional check should be introduced in StatePlannerBackgroundService after it resumes that would reassure that the state has actually started.

@ondryaso ondryaso added bug Something isn't working comp-states Related to the state management functionality low-priority A low-priority issue labels Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working comp-states Related to the state management functionality low-priority A low-priority issue
Projects
None yet
Development

No branches or pull requests

1 participant