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

TODO: Remove redundant validation #1091

Open
raman325 opened this issue Oct 10, 2024 · 0 comments
Open

TODO: Remove redundant validation #1091

raman325 opened this issue Oct 10, 2024 · 0 comments

Comments

@raman325
Copy link
Contributor

raman325 commented Oct 10, 2024

Z-Wave JS does a lot of internal validation that we have attempted to replicate within the library, and in some cases we may have been overly strict. Where it makes sense, we should fallback to the driver validation. To accomplish this we will need to make sure that whatever cases that lead to an exception and would also return an error from zwave-js are removed from the library.

CC @MartinHjelmare as this is something we have tackled here or there, but I think this is an overall design decision we should align on

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

No branches or pull requests

1 participant