Skip to content

Commit be96a3a

Browse files
Adam-MaciugaArekBalysNordic
authored andcommitted
bugfix: matter: weather_station: disable triggerEffect command
The Trigger Effect command on identify cluster was wrongly enabled disabled it to fit expectations Signed-off-by: Adam Maciuga <[email protected]>
1 parent 9faf080 commit be96a3a

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

applications/matter_weather_station/src/app_task.cpp

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,12 +99,17 @@ class IdentifyDelegateImplWeatherStation : public chip::app::Clusters::IdentifyD
9999
});
100100
}
101101

102+
/**
103+
* Trigger Effect command is not supported by default by this device
104+
* It can be enabled by returning true in the IsTriggerEffectEnabled function
105+
* When enabled handle the command in the function below
106+
*/
102107
void OnTriggerEffect(chip::app::Clusters::IdentifyCluster &cluster) override
103108
{
104109
Nrf::PostTask([] { BuzzerToggleState(); });
105110
}
106111

107-
bool IsTriggerEffectEnabled() const override { return true; }
112+
bool IsTriggerEffectEnabled() const override { return false; }
108113
};
109114

110115
IdentifyDelegateImplWeatherStation sIdentifyDelegateImplWeatherStation;

0 commit comments

Comments
 (0)