Skip to content

Add Tuya temp sensor _TZE200_w6n8jeuu variant #4019

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 29, 2025
Merged

Conversation

yucombinator
Copy link
Contributor

Add _TZE200_w6n8jeuu to list of Tuya TS0601 temperature sensors

Proposed change

Add a new Tuya device id to the TS0601 quirk.

_TZE200_w6n8jeuu is the device id I see in ZHA, but paring it with ZHA had no usable entities. Adding the device id to a local quirk file fixed the issue for me. After my change my sensor entities are detected and works:

Screenshot 2025-04-17 at 5 11 52 PM

Additional information

Link to the device on aliexpress: https://www.aliexpress.us/item/3256807102148050.html?spm=a2g0o.order_list.order_list_main.22.52b0180242jdlY&gatewayAdapt=glo2usa

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

Add _TZE200_w6n8jeuu
@yucombinator
Copy link
Contributor Author

I re-created the PR from #4016 but now based on the dev branch

Copy link

codecov bot commented Apr 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.17%. Comparing base (e4844d4) to head (aa6c1af).
Report is 6 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #4019   +/-   ##
=======================================
  Coverage   91.17%   91.17%           
=======================================
  Files         334      334           
  Lines       10858    10858           
=======================================
  Hits         9900     9900           
  Misses        958      958           

☔ 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.

@puddly puddly merged commit a941a89 into zigpy:dev Apr 29, 2025
9 checks passed
@TheJulianJES TheJulianJES changed the title Update tuya_sensor.py Add Tuya temp sensor _TZE200_w6n8jeuu variant Apr 29, 2025
@TheJulianJES
Copy link
Collaborator

TheJulianJES commented Apr 30, 2025

@yucombinator Can you send a full screenshot of the device page (including configuration entities) with the new v2 quirk? I'm wondering if this was perhaps added to the wrong base quirk, as the initial PR had it in a different spot.

Also, mind double checking that the local quirk you've used is the exact same as the patch here?

EDIT: This might be fully correct. I was confused by a similar Tuya sensor PR. I'd still appreciate a screenshot of the full device page in HA (including config entities) and you double checking that your custom quirk is the same is this patch.

@TheJulianJES TheJulianJES added the needs reviewer answer An answer from a reviewer is needed (e.g. why a PR isn't acceptable in the current state). label Apr 30, 2025
@TheJulianJES TheJulianJES mentioned this pull request Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs reviewer answer An answer from a reviewer is needed (e.g. why a PR isn't acceptable in the current state).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants