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

[modbus] Improve error message if data thing is configured for reading but not connected to a polling bridge. #16054

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

minichma
Copy link
Contributor

The fact that a modbus data thing can only be used for reading if it is connected to a polling bridge thing (rather than to a TCP or serial bridge thing), might not be obvious to every user. This PR improves the error message in case of a misconfiguration, i.e. if a thing is configured for reading but not connected to a polling bridge.

@minichma minichma requested a review from ssalonen as a code owner December 12, 2023 23:36
@minichma minichma force-pushed the modbus_data_write_errormessage branch from c72229d to ef5c549 Compare December 12, 2023 23:38
Copy link
Contributor

@ssalonen ssalonen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

…g but not connected to a polling bridge.

Signed-off-by: Markus Minichmayr <markus@minichmayr.net>
@minichma minichma force-pushed the modbus_data_write_errormessage branch from ef5c549 to 1d75029 Compare December 14, 2023 22:11
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you

@lolodomo lolodomo merged commit 03b7b1d into openhab:main Dec 14, 2023
3 checks passed
@lolodomo lolodomo added this to the 4.1 milestone Dec 14, 2023
@lolodomo lolodomo added the enhancement An enhancement or new feature for an existing add-on label Dec 14, 2023
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/openhab-4-1-milestone-discussion/149502/158

austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
…g but not connected to a polling bridge. (openhab#16054)

Signed-off-by: Markus Minichmayr <markus@minichmayr.net>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
joni1993 pushed a commit to joni1993/openhab-addons that referenced this pull request Oct 15, 2024
…g but not connected to a polling bridge. (openhab#16054)

Signed-off-by: Markus Minichmayr <markus@minichmayr.net>
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.

4 participants