-
-
Notifications
You must be signed in to change notification settings - Fork 425
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
[ephemeris] Evolutions of the Ephemeris module #1169
Conversation
Signed-off-by: Gaël L'hopital <[email protected]>
This pull request has been mentioned on openHAB Community. There might be relevant details there: |
Signed-off-by: Gaël L'hopital <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks you for the improvement. I left some comments.
.../org.openhab.core.model.script/src/org/eclipse/smarthome/model/script/actions/Ephemeris.java
Outdated
Show resolved
Hide resolved
...ts/src/main/java/org/eclipse/smarthome/core/ephemeris/internal/EphemerisManagerImplTest.java
Outdated
Show resolved
Hide resolved
...main/java/org/openhab/core/automation/internal/module/handler/EphemerisConditionHandler.java
Outdated
Show resolved
Hide resolved
...nhab.core.ephemeris/src/main/java/org/eclipse/smarthome/core/ephemeris/EphemerisManager.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Gaël L'hopital <[email protected]>
Signed-off-by: Gaël L'hopital <[email protected]>
...rc/main/java/org/eclipse/smarthome/core/ephemeris/internal/EphemerisManagerImplOSGiTest.java
Show resolved
Hide resolved
Signed-off-by: Gaël L'hopital <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good to me. Thanks.
Jollyday requires JAXB 2.3 on Java 11 whereas only 2.2 is available during compilation for Java 8 compatibility. See also: * https://github.com/openhab/openhab2-addons/issues/6334 * openhab#1170 * openhab#1169 Signed-off-by: Wouter Born <[email protected]>
Jollyday requires JAXB 2.3 on Java 11 whereas only 2.2 is available during compilation for Java 8 compatibility. See also: * https://github.com/openhab/openhab2-addons/issues/6334 * #1170 * #1169 Signed-off-by: Wouter Born <[email protected]>
* Evolutions of the Ephemeris module Signed-off-by: Gaël L'hopital <[email protected]> GitOrigin-RevId: 4fef010
Jollyday requires JAXB 2.3 on Java 11 whereas only 2.2 is available during compilation for Java 8 compatibility. See also: * https://github.com/openhab/openhab2-addons/issues/6334 * openhab#1170 * openhab#1169 Signed-off-by: Wouter Born <[email protected]> GitOrigin-RevId: d627a6b
This PR is a follow-up of this discussion
Added unit tests over Ephemeris
Ephemeris retrieves by default the Holiday key
Added a function to get localized name of a given holiday key
Added ability to get the number of days until a holiday
Added ability to get next first holiday
Adapted and extended scripting capability accordingly
Signed-off-by: Gaël L'hopital [email protected]