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

[tankerkoenig] Fixed Thing repeatedly going offline #8940

Merged
merged 3 commits into from
Nov 3, 2020
Merged

[tankerkoenig] Fixed Thing repeatedly going offline #8940

merged 3 commits into from
Nov 3, 2020

Conversation

JueBag
Copy link

@JueBag JueBag commented Nov 1, 2020

Fixes #8876
Fixes the problem caused by stations reporting "null" instead of "false"
Fixed faulty parsing of opening times in case of daily times ("täglich").

Signed-off-by: Jürgen Baginski opus42@gmx.de

Jürgen Baginski added 2 commits November 1, 2020 23:43
Incooperated check for 'null' iot to prevent NullPointerExceptions.

Signed-off-by: Jürgen Baginski <opus42@gmx.de>
Fixed false parsing of OpeningTimes.

Signed-off-by: Jürgen Baginski <opus42@gmx.de>
@kaikreuzer kaikreuzer added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Nov 2, 2020
@kaikreuzer
Copy link
Member

Build failed with spotless errors.

Signed-off-by: Jürgen Baginski <opus42@gmx.de>
@JueBag
Copy link
Author

JueBag commented Nov 2, 2020

Sorry for that one!

Actually this was my first try using VSCode, which worked for running the changed code in the IDE (able to use breakpoints!!). What didn't work last nigth was directly publishing the changes to Git, however that was working now.
AND two times publishing something and passing the DCO check immidiatly! That is a progress for me!

@J-N-K J-N-K added rebuild Triggers Jenkins PR build bug An unexpected problem or unintended behavior of an add-on and removed rebuild Triggers Jenkins PR build labels Nov 2, 2020
@JueBag
Copy link
Author

JueBag commented Nov 2, 2020

Is there anything that I should do? The build failed just BEFORE tankerkoenig.

@J-N-K
Copy link
Member

J-N-K commented Nov 2, 2020

That‘s why I retriggered the build. I do not understand why the new tado lib is a problem so often, since it is clearly available in the repo.

@kaikreuzer
Copy link
Member

Yeah, the tado lib issue is annoying, I couldn't yet figure out the reason for it either...

@kaikreuzer kaikreuzer added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Nov 2, 2020
@cpmeister cpmeister merged commit f8fd06e into openhab:main Nov 3, 2020
@cpmeister cpmeister added this to the 3.0.0.M3 milestone Nov 3, 2020
@kaikreuzer
Copy link
Member

@JueBag Please note that the issue title is what is used for the release notes later on - could you change it to something that roughly explains what situation is fixed?

kaikreuzer pushed a commit that referenced this pull request Nov 4, 2020
* [Tankerkoenig] Update StationHandler

Incooperated check for 'null' iot to prevent NullPointerExceptions.
* [Tankerkoenig] Updated WebserviceHandler

Fixed false parsing of OpeningTimes.
* Applied a spotless check   Signed-off-by: Jürgen Baginski <opus42@gmx.de>

Signed-off-by: Jürgen Baginski <opus42@gmx.de>
@kaikreuzer
Copy link
Member

Cherry-picked to 2.5.x.

@kaikreuzer kaikreuzer changed the title [Tankerkoenig] Fixes #8876 [tankerkoenig] Fixed Thing repeatedly going offline Dec 13, 2020
@kaikreuzer kaikreuzer modified the milestones: 3.0.0.M3, 2.5.11 Dec 13, 2020
nowaterman pushed a commit to nowaterman/openhab-addons that referenced this pull request Jan 19, 2021
* [Tankerkoenig] Update StationHandler

Incooperated check for 'null' iot to prevent NullPointerExceptions.
* [Tankerkoenig] Updated WebserviceHandler

Fixed false parsing of OpeningTimes.
* Applied a spotless check   Signed-off-by: Jürgen Baginski <opus42@gmx.de>
boehan pushed a commit to boehan/openhab-addons that referenced this pull request Apr 12, 2021
* [Tankerkoenig] Update StationHandler

Incooperated check for 'null' iot to prevent NullPointerExceptions.
* [Tankerkoenig] Updated WebserviceHandler

Fixed false parsing of OpeningTimes.
* Applied a spotless check   Signed-off-by: Jürgen Baginski <opus42@gmx.de>

Signed-off-by: Jürgen Baginski <opus42@gmx.de>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
* [Tankerkoenig] Update StationHandler

Incooperated check for 'null' iot to prevent NullPointerExceptions.
* [Tankerkoenig] Updated WebserviceHandler

Fixed false parsing of OpeningTimes.
* Applied a spotless check   Signed-off-by: Jürgen Baginski <opus42@gmx.de>

Signed-off-by: Jürgen Baginski <opus42@gmx.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on oh25x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[tankerkoenig] Thing repeatedly goes offline
4 participants