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

[time] Fix isBetweenTimes fails with time.toZDT not a function #151

Merged
merged 2 commits into from
Aug 17, 2022

Conversation

florian-h05
Copy link
Contributor

@rkoshak This is the PR I was talking about in the issue.

Fixes #150.
Previous PR #137 changed the way how toZDT is defined, which broke
isBetweenTimes.

Signed-off-by: Florian Hotze florianh_dev@icloud.com

Fixes openhab#150.
Previous PR openhab#137 changed the way how toZDT is defined, which broke
isBetweenTimes.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@florian-h05 florian-h05 added the bug Something isn't working label Aug 16, 2022
@florian-h05 florian-h05 added this to the 2.x.x milestone Aug 16, 2022
@florian-h05 florian-h05 added the workinprogress Work in Progress — not completed or tested label Aug 16, 2022
Copy link
Contributor

@rkoshak rkoshak left a comment

Choose a reason for hiding this comment

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

I tested it and it works for me. Thanks for the quick fix!

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@florian-h05 florian-h05 removed the workinprogress Work in Progress — not completed or tested label Aug 17, 2022
@florian-h05 florian-h05 merged commit 5be991d into openhab:main Aug 17, 2022
@florian-h05 florian-h05 deleted the issue-150 branch August 17, 2022 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

time.isBetweenTimes cannot see time.toZDT()
2 participants