Skip to content
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

Update tuya.ts #7072

Merged
merged 2 commits into from
Feb 15, 2024
Merged

Update tuya.ts #7072

merged 2 commits into from
Feb 15, 2024

Conversation

RafaelDuncan
Copy link
Contributor

adding support for manufacturerName: '_TZE204_5cuocqty' TS0601
merged with the existing TS0601_dimmer_5

RafaelDuncan and others added 2 commits February 14, 2024 22:53
adding support for manufacturerName: '_TZE204_5cuocqty'
TS0601
@Koenkk Koenkk mentioned this pull request Feb 15, 2024
@Koenkk Koenkk merged commit 299b7c3 into Koenkk:master Feb 15, 2024
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Feb 15, 2024

thanks!

@RafaelDuncan
Copy link
Contributor Author

RafaelDuncan commented Feb 28, 2024

hello @Koenkk ,

Today I discovered that this dimmer module (_TZE204_5cuocqty' TS0601) also has the function for choosing the "switch type". This way you can indicate whether a toggle or momentary switch is connected. this is comparable to module _TZ3210_3mpwqzuu. Should I submit a change request for this? otherwise this function cannot be used.

I looked up the Tuya Datapoint via the Tuya gateway and see that the SwitchType datapoint has "57". I find it difficult to merge this with the T0601_dimmer_5 as I don't really know how to enter the other variables on this line. could you help me with this?

UPDATE: I have just tried to edit the file... if all goes well, it is now submitted for review.

RafaelDuncan referenced this pull request in RafaelDuncan/zigbee-herdsman-converters Feb 28, 2024
Today I discovered that this dimmer module (_TZE204_5cuocqty' TS0601) also has the function for choosing the "switch type". This way you can indicate whether a toggle or momentary switch is connected. this is comparable to module _TZ3210_3mpwqzuu. Should I submit a change request for this? otherwise this function cannot be used.

I looked up the Tuya Datapoint via the Tuya gateway and see that the SwitchType datapoint has "57". I find it difficult to merge this with the T0601_dimmer_5 as I don't really know how to enter the other variables on this line. could you help me with this?
@Koenkk
Copy link
Owner

Koenkk commented Feb 29, 2024

Can you create a new pull request?

@RafaelDuncan
Copy link
Contributor Author

@Koenkk did you see request
RafaelDuncan@3d43ed3

@RafaelDuncan
Copy link
Contributor Author

RafaelDuncan commented Feb 29, 2024

@Koenkk did you see request
RafaelDuncan@3d43ed3

@Koenkk
Copy link
Owner

Koenkk commented Feb 29, 2024

Yes, but this is not a pull request, can you create it? Alternatively I can add the changes directly.

@RafaelDuncan
Copy link
Contributor Author

Yes, but this is not a pull request, can you create it? Alternatively I can add the changes directly.

I don't know if I will be able to do that... otherwise could you make the change yourself?

@Koenkk
Copy link
Owner

Koenkk commented Feb 29, 2024

Done!

@RafaelDuncan
Copy link
Contributor Author

thank you very much Koenkk

@Flight-Lab
Copy link

switchType does not seem to be working with '_TZE204_dcnsggvz', which is another fingerprint under the TS0601_dimmer_5 model.

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