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

[boschshc] Release v1.1 #10097

Merged
merged 13 commits into from Feb 13, 2021
Merged

[boschshc] Release v1.1 #10097

merged 13 commits into from Feb 13, 2021

Conversation

coeing
Copy link
Contributor

@coeing coeing commented Feb 7, 2021

Hi there,

Hope all of you had a good start into the new year :)

We prepared a patch version of our binding, as there were some severe bugs in it which stopped the binding to work after 24 hours and after a restart of the controller.

Plus we use the new version to be up-to-date with the Bosch Rest API (v2.1) and fixed two minor issues (binding title and units used).

Here is the list of changes in this version:

@coeing coeing marked this pull request as ready for review February 7, 2021 20:04
GerdZanker and others added 12 commits February 7, 2021 21:06
…y values

all other QuantityTypes in bindingcode are fine

Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
Replaced the SHC occurrences with Smart Home,
to avoid technical names.

Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
… offline

Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
… of the previous long polling http request

This might be the reason why the subscribe request does never finish or finishes with a timeout

Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
…ot get timeout from HTTP client

Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
We need to try to reconnect again and again (with 15 seconds between the requests) as the controller may have been restarted (update, manual restart,...). This is already done by the initial access, so I reuse that mechanism.

Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
Co-authored-by: Gerd Zanker <gerd.zanker@web.de>
…out failed long polling

Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
… is handled now

Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
each HTTP request will use now the defined "avp-version=2.1" for request to the smart home controller

Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
removed the old static version string
access OSGi bundle version information instead

Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
- added isOnline check and isAccessPossible now failed in case HTTPStatus is an error
- same HTTPStatus check done to all blocking send() request calls
- using i18n strings for all bridge updateStatus calls
- skipped the 'controller' and use only 'Bosch Smart Home' in descriptions
- added more @nullable annotations

Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
@coeing coeing changed the title Release v1.1 [boschshc] Release v1.1 Feb 8, 2021
@Hilbrand Hilbrand added bug An unexpected problem or unintended behavior of an add-on enhancement An enhancement or new feature for an existing add-on labels Feb 8, 2021
Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
@cpmeister cpmeister merged commit a796e47 into openhab:main Feb 13, 2021
@cpmeister cpmeister removed the bug An unexpected problem or unintended behavior of an add-on label Feb 13, 2021
@cpmeister cpmeister added this to the 3.1 milestone Feb 13, 2021
@coeing coeing deleted the releases/v1.1 branch February 14, 2021 19:56
themillhousegroup pushed a commit to themillhousegroup/openhab2-addons that referenced this pull request May 10, 2021
* openhab#72 changed use units of measure for the twinguard humidity and purity values

all other QuantityTypes in bindingcode are fine

* openhab#77 changed title of binding to Bosch Smart Home

Replaced the SHC occurrences with Smart Home,
to avoid technical names.

* openhab#62 Try to restart long polling when it fails before taking the thing offline

* openhab#62 Run subscribe request on a new thread instead of using the thread of the previous long polling http request

This might be the reason why the subscribe request does never finish or finishes with a timeout

* openhab#74 Run the whole long polling response handling in a new thread to not get timeout from HTTP client

* openhab#74 Schedule initial access when long polling fails unexpected

We need to try to reconnect again and again (with 15 seconds between the requests) as the controller may have been restarted (update, manual restart,...). This is already done by the initial access, so I reuse that mechanism.

* Use direct formatting of logger.trace instead of String.format

* openhab#76 Use i18n texts instead of raw translations for status messages about failed long polling

* openhab#76 Use logger.debug instead of logger.warn for long poll error as it is handled now

* openhab#78 defined api-version

each HTTP request will use now the defined "avp-version=2.1" for request to the smart home controller

* logging bundle version

removed the old static version string
access OSGi bundle version information instead

* openhab#75 improved initial access

- added isOnline check and isAccessPossible now failed in case HTTPStatus is an error
- same HTTPStatus check done to all blocking send() request calls
- using i18n strings for all bridge updateStatus calls
- skipped the 'controller' and use only 'Bosch Smart Home' in descriptions
- added more @nullable annotations
* added newline

Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
Signed-off-by: John Marshall <john.marshall.au@gmail.com>
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
* openhab#72 changed use units of measure for the twinguard humidity and purity values

all other QuantityTypes in bindingcode are fine

* openhab#77 changed title of binding to Bosch Smart Home

Replaced the SHC occurrences with Smart Home,
to avoid technical names.

* openhab#62 Try to restart long polling when it fails before taking the thing offline

* openhab#62 Run subscribe request on a new thread instead of using the thread of the previous long polling http request

This might be the reason why the subscribe request does never finish or finishes with a timeout

* openhab#74 Run the whole long polling response handling in a new thread to not get timeout from HTTP client

* openhab#74 Schedule initial access when long polling fails unexpected

We need to try to reconnect again and again (with 15 seconds between the requests) as the controller may have been restarted (update, manual restart,...). This is already done by the initial access, so I reuse that mechanism.

* Use direct formatting of logger.trace instead of String.format

* openhab#76 Use i18n texts instead of raw translations for status messages about failed long polling

* openhab#76 Use logger.debug instead of logger.warn for long poll error as it is handled now

* openhab#78 defined api-version

each HTTP request will use now the defined "avp-version=2.1" for request to the smart home controller

* logging bundle version

removed the old static version string
access OSGi bundle version information instead

* openhab#75 improved initial access

- added isOnline check and isAccessPossible now failed in case HTTPStatus is an error
- same HTTPStatus check done to all blocking send() request calls
- using i18n strings for all bridge updateStatus calls
- skipped the 'controller' and use only 'Bosch Smart Home' in descriptions
- added more @nullable annotations
* added newline

Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
* openhab#72 changed use units of measure for the twinguard humidity and purity values

all other QuantityTypes in bindingcode are fine

* openhab#77 changed title of binding to Bosch Smart Home

Replaced the SHC occurrences with Smart Home,
to avoid technical names.

* openhab#62 Try to restart long polling when it fails before taking the thing offline

* openhab#62 Run subscribe request on a new thread instead of using the thread of the previous long polling http request

This might be the reason why the subscribe request does never finish or finishes with a timeout

* openhab#74 Run the whole long polling response handling in a new thread to not get timeout from HTTP client

* openhab#74 Schedule initial access when long polling fails unexpected

We need to try to reconnect again and again (with 15 seconds between the requests) as the controller may have been restarted (update, manual restart,...). This is already done by the initial access, so I reuse that mechanism.

* Use direct formatting of logger.trace instead of String.format

* openhab#76 Use i18n texts instead of raw translations for status messages about failed long polling

* openhab#76 Use logger.debug instead of logger.warn for long poll error as it is handled now

* openhab#78 defined api-version

each HTTP request will use now the defined "avp-version=2.1" for request to the smart home controller

* logging bundle version

removed the old static version string
access OSGi bundle version information instead

* openhab#75 improved initial access

- added isOnline check and isAccessPossible now failed in case HTTPStatus is an error
- same HTTPStatus check done to all blocking send() request calls
- using i18n strings for all bridge updateStatus calls
- skipped the 'controller' and use only 'Bosch Smart Home' in descriptions
- added more @nullable annotations
* added newline

Signed-off-by: Gerd Zanker <gerd.zanker@web.de>
Signed-off-by: Christian Oeing <christian.oeing@slashgames.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants