Skip to content

Conversation

@nyok92
Copy link
Contributor

@nyok92 nyok92 commented Sep 25, 2025

Update of the Ceres plus petfeeder config

@github-project-automation github-project-automation bot moved this to 👀 In review in Tuya Local Sep 26, 2025
@make-all
Copy link
Owner

This is quite a major change. Some of it appears to just be reordering of entities, which adds to the noise and makes it difficult to find the actual changes amongst it.

At least one dp has had its type changed, dp 246 has switched from string (as documented in #507 and most likely correct) to integer, although #507 is a bit confused about the purpose of the dp.

Changes that affect detection like that will need some evidence with them to convince me that it is a good change. Also other changes that affect entity naming need good justification, as that will affect other users who have the config installed also, so just "I like this name better" is not likely to be enough.

@make-all make-all added the awaiting feedback Needs more information label Oct 4, 2025
@make-all make-all moved this from 👀 In review to 🏗 Stalled in Tuya Local Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting feedback Needs more information

Projects

Status: 🏗 Stalled

Development

Successfully merging this pull request may close these issues.

2 participants