Skip to content

Conversation

@nickolas-deboom
Copy link
Contributor

Remove the GoToTiltPercentage command sent when the presetPosition capability command is received, which previously would set the tilt preset level to a default level of 50%. A new capability will be introduced later to support setting the tilt preset level.

local endpoint_id = device:component_to_endpoint(cmd.component)
local lift_eps = device:get_endpoints(clusters.WindowCovering.ID, {feature_bitmap = clusters.WindowCovering.types.Feature.LIFT})
local tilt_eps = device:get_endpoints(clusters.WindowCovering.ID, {feature_bitmap = clusters.WindowCovering.types.Feature.TILT})
if #lift_eps > 0 then
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note that this check isn't needed since any device using a profile with the windowShadePresetcapability supports the LIFT feature.

@github-actions
Copy link

github-actions bot commented Nov 18, 2025

Channel deleted.

@github-actions
Copy link

github-actions bot commented Nov 18, 2025

Test Results

   71 files    469 suites   0s ⏱️
2 457 tests 2 457 ✅ 0 💤 0 ❌
4 177 runs  4 177 ✅ 0 💤 0 ❌

Results for commit 631dc6c.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Nov 18, 2025

File Coverage
All files 74%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-window-covering/src/matter-window-covering-position-updates-while-moving/init.lua 36%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-window-covering/src/init.lua 90%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 3158029

Remove the default tilt preset level of 50%, which is not ideal for user
experience.
@nickolas-deboom nickolas-deboom force-pushed the matter-window-covering-remove-default-tilt-preset branch from 631dc6c to 3158029 Compare November 19, 2025 21:06
@nickolas-deboom nickolas-deboom merged commit e628b47 into main Nov 19, 2025
9 checks passed
@nickolas-deboom nickolas-deboom deleted the matter-window-covering-remove-default-tilt-preset branch November 19, 2025 21:08
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.

4 participants