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

[paradoxalarm] Discovery service implementation #5826

Merged
merged 4 commits into from
Jul 23, 2019
Merged

[paradoxalarm] Discovery service implementation #5826

merged 4 commits into from
Jul 23, 2019

Conversation

theater
Copy link
Contributor

@theater theater commented Jul 14, 2019

Paradox alarm - Discovery service implementation

[paradoxalarm] Implementation of discovery service for entities. Bridge must be configured. When it establishes communication with security system, the discovery service can gather all entities (partitions, zones and panel info)

Signed-off-by: Konstantin Polihronov [email protected]

Signed-off-by: Konstantin Polihronov <[email protected]>
@TravisBuddy
Copy link

Travis tests were successful

Hey @theater,
we found no major flaws with your code. Still you might want to look at this logfile, as we usually suggest some optional improvements.

@theater theater changed the title Paradox alarm - Discovery service implementation [paradoxalarm] - Discovery service implementation Jul 15, 2019
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.

Just two little remarks

* Changed final map in handler factory to non-final
* Removed runtime exception in discovery service implementation logging
it at DEBUG level plus trace

Signed-off-by: Konstantin Polihronov <[email protected]>
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.
Maybe we should wait for a valid build.

Copy link
Contributor

@cweitkamp cweitkamp left a comment

Choose a reason for hiding this comment

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

Thank you. I added some minor comments.

Copy link
Contributor

@cweitkamp cweitkamp left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@cweitkamp cweitkamp merged commit 68e0536 into openhab:master Jul 23, 2019
@wborn wborn added this to the 2.5 milestone Jul 30, 2019
@cweitkamp cweitkamp added the enhancement An enhancement or new feature for an existing add-on label Aug 3, 2019
sprehn pushed a commit to sprehn/openhab-addons that referenced this pull request Aug 21, 2019
* Discovery service implementation

Signed-off-by: Konstantin Polihronov <[email protected]>
@theater theater deleted the ParadoxAlarmDiscoveryService branch August 31, 2019 07:06
ne0h pushed a commit to ne0h/openhab-addons that referenced this pull request Sep 15, 2019
* Discovery service implementation

Signed-off-by: Konstantin Polihronov <[email protected]>
Signed-off-by: Maximilian Hess <[email protected]>
@wborn wborn changed the title [paradoxalarm] - Discovery service implementation [paradoxalarm] Discovery service implementation Dec 8, 2019
@wborn wborn removed the enhancement An enhancement or new feature for an existing add-on label Dec 8, 2019
Pshatsillo pushed a commit to Pshatsillo/openhab-addons that referenced this pull request Dec 8, 2019
* Discovery service implementation

Signed-off-by: Konstantin Polihronov <[email protected]>
tmrobert8 pushed a commit to tmrobert8/openhab-addons that referenced this pull request Jan 21, 2020
* Discovery service implementation

Signed-off-by: Konstantin Polihronov <[email protected]>
Signed-off-by: Tim Roberts <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants