Skip to content

[hue] Fix firmware availability issues #20709

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.

Sign up for GitHub

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

Merged
jlaur merged 7 commits into from
May 16, 2026
Merged

[hue] Fix firmware availability issues #20709

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
a1f02b4
fix software update trigger issues
andrewfg May 12, 2026
5edad68
various
andrewfg May 12, 2026
266a258
bug fix
andrewfg May 12, 2026
9f53423
fix update instructions
andrewfg May 12, 2026
27024a4
Merge remote-tracking branch 'upstream/main' into hue-software-fix
andrewfg May 12, 2026
cf2334d
Potential fix for pull request finding
andrewfg May 12, 2026
a23dfbb
fix bug
andrewfg May 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion ...enhab.binding.hue/src/main/java/org/openhab/binding/hue/internal/HueBindingConstants.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@
*/
package org.openhab.binding.hue.internal;

import static org.openhab.core.thing.ChannelUID.CHANNEL_GROUP_SEPARATOR;

import java.util.Map;
import java.util.Set;

Expand Down Expand Up @@ -145,6 +147,8 @@ public class HueBindingConstants {
public static final ThingTypeUID THING_TYPE_AREA = new ThingTypeUID(BINDING_ID, "area");
public static final ThingTypeUID THING_TYPE_SERVICE_GROUP = new ThingTypeUID(BINDING_ID, "service-group");

public static final String CHANNEL_GROUP_SOFTWARE = "software";

// channels for API v2
public static final String CHANNEL_2_COLOR = CHANNEL_COLOR;
public static final String CHANNEL_2_COLOR_TEMP_PERCENT = "color-temperature";
Expand Down Expand Up @@ -188,7 +192,8 @@ public class HueBindingConstants {
public static final String CHANNEL_2_MOTION_AREA_ENABLED = "motion-area-enabled";
public static final String CHANNEL_2_SECURITY_MOTION = "security-motion";
public static final String CHANNEL_2_SECURITY_MOTION_LAST_UPDATED = "security-motion-last-updated";
public static final String CHANNEL_2_UPDATE_READY_TO_INSTALL = "update-ready";
public static final String CHANNEL_2_UPDATE_READY_TO_INSTALL = //
Comment thread
andrewfg marked this conversation as resolved.
CHANNEL_GROUP_SOFTWARE + CHANNEL_GROUP_SEPARATOR + "update-ready";

// channel IDs that (optionally) support dynamics
public static final Set<String> DYNAMIC_CHANNELS = Set.of(CHANNEL_2_BRIGHTNESS, CHANNEL_2_COLOR,
Expand Down
7 changes: 6 additions & 1 deletion ...inding.hue/src/main/java/org/openhab/binding/hue/internal/handler/Clip2BridgeHandler.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,7 @@ public class Clip2BridgeHandler extends BaseBridgeHandler {
private Map<Integer, Future<?>> resourcesEventTasks = new ConcurrentHashMap<>();

private boolean assetsLoaded;
private boolean softwareUpdateReadyNotificationSent;
Comment thread
andrewfg marked this conversation as resolved.
Outdated
private int applKeyRetriesRemaining;
private int connectRetriesRemaining;

Expand Down Expand Up @@ -517,6 +518,7 @@ public void initialize() {
applKeyRetriesRemaining = APPLICATION_KEY_MAX_TRIES;
connectRetriesRemaining = RECONNECT_MAX_TRIES;
initializeAssets();
softwareUpdateReadyNotificationSent = false;
}

/**
Expand Down Expand Up @@ -959,7 +961,10 @@ private UpdateStatusV2 refreshSoftwareStatusUI() {
updateStatus(ThingStatus.ONLINE, ThingStatusDetail.NONE, description);
break;
case READY_TO_INSTALL:
triggerChannel(CHANNEL_2_UPDATE_READY_TO_INSTALL);
if (!softwareUpdateReadyNotificationSent) {
softwareUpdateReadyNotificationSent = true;
triggerChannel(CHANNEL_2_UPDATE_READY_TO_INSTALL);
}
Comment thread
andrewfg marked this conversation as resolved.
// note: fall through to set Thing status, status detail, and status description
case INSTALL_FAILED:
default:
Expand Down
Loading