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

Add NovaDigital SL-06 smart lock #1830

Merged
merged 18 commits into from
Apr 20, 2024
Merged

Add NovaDigital SL-06 smart lock #1830

merged 18 commits into from
Apr 20, 2024

Conversation

rudironsonijr
Copy link
Contributor

ThomasADavis pushed a commit to ThomasADavis/tuya-local that referenced this pull request Apr 15, 2024
History: this config was originally made with `invert: true` at dps level,
where the desired effect was not produced. After yaml lint tests were added,
this issue was detected and "fixed" by moving it into a `mapping`.  However,
the original presence of that `invert: true` seems to have been a mistake, the
values should not be inverted.

Discussion make-all#1830
New configs should not define entities that are already deprecated.
@make-all make-all merged commit cb55caf into make-all:main Apr 20, 2024
4 checks passed
@rudironsonijr rudironsonijr deleted the feature/novadigital_sl06_lock branch April 20, 2024 12:01
timlaing pushed a commit to timlaing/tuya-local that referenced this pull request Aug 8, 2024
History: this config was originally made with `invert: true` at dps level,
where the desired effect was not produced. After yaml lint tests were added,
this issue was detected and "fixed" by moving it into a `mapping`.  However,
the original presence of that `invert: true` seems to have been a mistake, the
values should not be inverted.

Discussion make-all#1830
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants