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

[hue] Improved code (notify status listener) #7511

Merged
merged 2 commits into from
May 2, 2020

Conversation

lolodomo
Copy link
Contributor

@lolodomo lolodomo commented May 1, 2020

Signed-off-by: Laurent Garnier [email protected]

@lolodomo lolodomo requested a review from cweitkamp as a code owner May 1, 2020 10:17

private static final String DEVICE_TYPE = "EclipseSmartHome";

private static enum STATUS_TYPE {
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change enum name to camelcase

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right. It is fixed.

Signed-off-by: Laurent Garnier <[email protected]>
Copy link
Contributor

@cpmeister cpmeister left a comment

Choose a reason for hiding this comment

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

LGTM

@lolodomo
Copy link
Contributor Author

lolodomo commented May 2, 2020

If you could merge this one, I will then update my PR about the hue groups because this will introduce a confliuct.

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. I left on minor comment - not related to your changes.

private long sensorPollingInterval = TimeUnit.MILLISECONDS.toMillis(500);
public static final Set<ThingTypeUID> SUPPORTED_THING_TYPES = Collections.singleton(THING_TYPE_BRIDGE);

private static final String DEVICE_TYPE = "EclipseSmartHome";
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it is time to change this to "openHAB".

@cweitkamp cweitkamp self-requested a review May 2, 2020 15:05
@cweitkamp cweitkamp merged commit cc0f769 into openhab:2.5.x May 2, 2020
@lolodomo lolodomo deleted the hue_improve1 branch May 2, 2020 15:16
@cpmeister cpmeister added this to the 2.5.5 milestone May 2, 2020
MrRonfo pushed a commit to MrRonfo/openhab-addons that referenced this pull request May 25, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
Signed-off-by: Fabio Possieri <[email protected]>
LoungeFlyZ pushed a commit to LoungeFlyZ/openhab2-addons that referenced this pull request Jun 8, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
J-N-K pushed a commit to J-N-K/openhab-addons that referenced this pull request Jul 14, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
CSchlipp pushed a commit to CSchlipp/openhab-addons that referenced this pull request Jul 26, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
Signed-off-by: CSchlipp <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
DaanMeijer pushed a commit to DaanMeijer/openhab-addons that referenced this pull request Sep 1, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[email protected]>
Signed-off-by: Daan Meijer <[email protected]>
markus7017 pushed a commit to markus7017/openhab-addons that referenced this pull request Sep 19, 2020
* [hue] Improved code (notify status listener)

Signed-off-by: Laurent Garnier <[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.

3 participants