Skip to content

Conversation

stefankv93
Copy link

Proposed change

The repo is missing a quirk for Aqara T1 led strip light:
https://www.aqara.com/eu/product/led-strip-t1/

I recently got this device, hence got some time to develop a quirk for it.

Additional information

Fixes #2886

I tested the quirk with the custom quirk applied on my local home assistant instance:
image

Device diagnostics

Attaching device diagnostics with the quirk applied
zha-01JC6W4XYGYJJFR3D9AJJAQXJH-Aqara Led Strip T1-eb15b407c240b5e4f017fdf4f4798451.json

Checklist

  • The changes are tested and work correctly
  • pre-commit checks pass / the code has been formatted using Black
  • Tests have been added to verify that the new code works
  • Device diagnostics data has been attached

Copy link

codecov bot commented Aug 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.16%. Comparing base (6213bef) to head (e7257eb).
⚠️ Report is 25 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #4274      +/-   ##
==========================================
+ Coverage   92.05%   92.16%   +0.11%     
==========================================
  Files         350      359       +9     
  Lines       11601    11970     +369     
==========================================
+ Hits        10679    11032     +353     
- Misses        922      938      +16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TheJulianJES TheJulianJES changed the title Adding a quirk for Aqara T1 led strip Add Aqara T1 led strip quirk Aug 24, 2025
@TheJulianJES TheJulianJES added needs review This PR should be reviewed soon, as it generally looks good. Xiaomi Request/PR regarding a Xiaomi device labels Aug 24, 2025
@ahyattdev
Copy link

Just tested this to correct the length and it worked! I have one Aqara T1 LED Strip (2m) + 1 extension (1m). The length defaulted to 2, and I set it to 3. Now the whole strip lights up!

@stefankv93
Copy link
Author

Just tested this to correct the length and it worked! I have one Aqara T1 LED Strip (2m) + 1 extension (1m). The length defaulted to 2, and I set it to 3. Now the whole strip lights up!

I added also a couple of other supported options, for example, you can activate and configure audio mode, where lights changes in sync with sound.

Hi @TheJulianJES, is there anything needed in this PR to get this in?

@Icecovery
Copy link

Hi @stefankv93, I made a few changes to this PR at stefankv93#1, namely:

  • Changed preset range from 1-32 to 0-8, as only these values work on my T1 LED Strip.
  • Added human-readable names for the presets.

Please review and let me know if you have any questions or concerns. Thanks!

@stefankv93
Copy link
Author

Hi @stefankv93, I made a few changes to this PR at stefankv93#1, namely:

  • Changed preset range from 1-32 to 0-8, as only these values work on my T1 LED Strip.
  • Added human-readable names for the presets.

Please review and let me know if you have any questions or concerns. Thanks!

Thanks for catching this, this is great! I included your commit changes in this PR. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review This PR should be reviewed soon, as it generally looks good. Xiaomi Request/PR regarding a Xiaomi device
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Device Support Request] Aqara T1 LED Strip
4 participants