Skip to content

Add initialization command for BLE-YL01#9681

Merged
Koenkk merged 4 commits intoKoenkk:masterfrom
jl1990:master
Jul 25, 2025
Merged

Add initialization command for BLE-YL01#9681
Koenkk merged 4 commits intoKoenkk:masterfrom
jl1990:master

Conversation

@jl1990
Copy link
Copy Markdown
Contributor

@jl1990 jl1990 commented Jul 24, 2025

BLE-YL01 seems to need this attribute set in order to trigger constant updates.
This should be tested properly before being merged, but in my experience after setting up the device with this command, I keep receiving updates after some hours (although the frequency as I mention is not ideal, it can take multiple hours before triggering an update).

Copilot AI review requested due to automatic review settings July 24, 2025 19:21
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds an initialization command for the BLE-YL01 device to enable constant updates by setting a specific attribute during device configuration.

Key Changes

  • Replaces the simple tuya.configureMagicPacket configuration with a custom async function
  • Adds a write operation to set attribute 65502 with value 0x13 on the genBasic cluster
Comments suppressed due to low confidence (3)

@jl1990 jl1990 changed the title Add initialization command for BLE-YL01 [Do not merge till this is tested properly] Add initialization command for BLE-YL01 Jul 24, 2025
@Koenkk Koenkk marked this pull request as draft July 25, 2025 15:17
@Koenkk
Copy link
Copy Markdown
Owner

Koenkk commented Jul 25, 2025

This should be tested properly before being merged,

Are you going to test this more? I see little risk in merging this PR, in most cases it doesn't work anyway so there is nothing to break (and probably this won't break it even it it is working)

@jl1990
Copy link
Copy Markdown
Contributor Author

jl1990 commented Jul 25, 2025

This should be tested properly before being merged,

Are you going to test this more? I see little risk in merging this PR, in most cases it doesn't work anyway so there is nothing to break (and probably this won't break it even it it is working)

I was expecting someone else to provide some feedback not to depend just on my device.
But as I guess the idea is to merge it to the dev branch and currently this device is not working properly it should be fine.
Feel free to merge it.

@jl1990 jl1990 changed the title [Do not merge till this is tested properly] Add initialization command for BLE-YL01 Add initialization command for BLE-YL01 Jul 25, 2025
@Koenkk Koenkk marked this pull request as ready for review July 25, 2025 17:27
@Koenkk Koenkk merged commit c097329 into Koenkk:master Jul 25, 2025
3 checks passed
@Koenkk
Copy link
Copy Markdown
Owner

Koenkk commented Jul 25, 2025

Thanks!

SimplaHome pushed a commit to SimplaHome/zigbee-herdsman-converters that referenced this pull request Aug 15, 2025
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