-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[tibber] Rework binding (channel groups, UoM, Thing actions and more) #18726
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
Merged
Merged
Changes from 26 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
11b23ab
use common httpclient
weymann d249a20
stability rework initial version
weymann a7ffca3
move to cron scheduling
weymann 89abe99
version update
weymann 4c2de8b
thing actions
weymann b8e1067
v0.2-alpha
weymann 3c3e1bb
OH5 header changes
weymann 6f6498d
full example
weymann 4c31562
full example
weymann f8abb57
update instructions
weymann c916515
add level to PriceInfo
weymann dd8e50f
thing status translations
weymann 11a140f
header check
weymann 8fe4836
checkstyle
weymann 1409641
junit test cleanup
weymann 45e3e27
compiler warnings
weymann a368355
optimize message handling
weymann 7b5af91
JSON error handling
weymann 4efb2c7
readme
weymann 152a9c1
readme
weymann 7701800
readme
weymann e7405ce
spot price update
weymann eb56a1c
fix channel typo
weymann 68002f4
fix obvious review findings
weymann faadc5a
further review findings
weymann 9bc6f62
Use HTTP status constant
jlaur 954d21a
Remove undocumented values
jlaur 49c688d
Extend warning
jlaur 9ebfb53
Remove spaces before colons
jlaur f2f7c09
Reapply semantic tags
jlaur 72cf430
Revert indirection
jlaur File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
For future refactoring it would be a step forward to create proper DTO's for these
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.
See #18726 (comment)