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
[sitemap] Extend chart periods to cover past and future #4172
Conversation
Closes openhab/openhab-webui#2518 Signed-off-by: Laurent Garnier <lg.hc@free.fr>
7ade1e1
to
fb5f7c1
Compare
I added unit tests, this is now ready for review. |
The change is fully backward compatible. |
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.
@lolodomo many thanks for this. I did not test it yet on a live system. But in the meantime I do have one minor comment.
...ab.core.ui/src/test/java/org/openhab/core/ui/internal/chart/ChartServletPeriodParamTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
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.
lgtm, thanks!
Chart widget sitemap configuration has been extended with support for future periods in openhab/openhab-core#4172. This adds configuration in the UI. See #2518. Signed-off-by: Mark Herwege <mark.herwege@telenet.be>
Closes openhab/openhab-webui#2518
Signed-off-by: Laurent Garnier lg.hc@free.fr