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
switched PersistenceExtensions to ZonedDateTime #1588
Conversation
Signed-off-by: Kai Kreuzer <kai@openhab.org>
Signed-off-by: Kai Kreuzer <kai@openhab.org>
Signed-off-by: Kai Kreuzer <kai@openhab.org>
Signed-off-by: Kai Kreuzer <kai@openhab.org>
@openhab/core-maintainers Tests are finally green, so ready for review and hopefully soon to merge! |
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.
In general it looks good to me and I only have a couple of questions/remarks:
...re.model.script/src/org/openhab/core/model/script/scoping/ScriptImplicitlyImportedTypes.java
Show resolved
Hide resolved
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!
Looks like the webui's also need to be updated for these changes @kaikreuzer. |
Yeah, indeed, the CometVisu seems to have issues now... |
Signed-off-by: Kai Kreuzer <kai@openhab.org> GitOrigin-RevId: 3c83e8a
In #1451 I changed the rules from
Instant
toZonedDateTime
, but I had missed thePersistenceExtensions
. Fixing this, I also did some further clean up and also moved it tocore.persistence
as it does not have any dependency on the persist DSL itself.Signed-off-by: Kai Kreuzer kai@openhab.org