Skip to content

Fix: Emit button event 1002 again for TRADFRI remote control after mode switch - #4951

Merged
manup merged 1 commit into
dresden-elektronik:masterfrom
SwoopX:tradfri
Jun 13, 2021
Merged

Fix: Emit button event 1002 again for TRADFRI remote control after mode switch#4951
manup merged 1 commit into
dresden-elektronik:masterfrom
SwoopX:tradfri

Conversation

@SwoopX

@SwoopX SwoopX commented May 30, 2021

Copy link
Copy Markdown
Collaborator

When pushing the on/off button of the switch approx. 5 seconds, the device changes the operational mode so that the arrow buttons do not send any commands on hold/long press. Under certain conditions (apparently when the arrow keys haven't been used before), this can lead to button event 1002 not being emitted, although the corresponding command is send.

As the respective code was intended for the STYRBAR remote, it is now restricted to that device.

@SwoopX SwoopX added this to the v2.12.1-beta milestone May 30, 2021
@SwoopX SwoopX linked an issue May 30, 2021 that may be closed by this pull request
@SwoopX SwoopX changed the title Emit button event 1002 again for TRADFRI remote control after mode switch FIX: Emit button event 1002 again for TRADFRI remote control after mode switch May 30, 2021
@SwoopX SwoopX changed the title FIX: Emit button event 1002 again for TRADFRI remote control after mode switch Fix: Emit button event 1002 again for TRADFRI remote control after mode switch May 30, 2021
@manup
manup merged commit 29ec9ff into dresden-elektronik:master Jun 13, 2021
@SwoopX
SwoopX deleted the tradfri branch June 14, 2021 16:59
@Nindaleth

Copy link
Copy Markdown

Could you please fix a typo related to this PR in the changelogs? Both English and German changelogs list this fix as #5951 instead of #4951 which gives 404 instead of the proper thing. Thanks!

@SwoopX

SwoopX commented Jul 10, 2021

Copy link
Copy Markdown
Collaborator Author

Pinged @manup

@manup

manup commented Jul 10, 2021

Copy link
Copy Markdown
Member

Thanks for the hint, link is corrected now.

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.

"No button handler for: TRADFRI remote control"

3 participants