We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I think this should be powerlevel, not powerLevel?
powerlevel
powerLevel
zwave-js-ui/src/stores/base.js
Line 28 in 1741fac
Referenced here (I think):
zwave-js-ui/lib/ZwaveClient.ts
Lines 2137 to 2143 in 1741fac
I don't think it's causing an issue if you change settings via the UI, but something I noticed.
The text was updated successfully, but these errors were encountered:
Both should be powerLevel, not powerlevel as u want to stay with camelcase! https://google.github.io/styleguide/tsguide.html#identifiers
But I think u have a point in reporting inconsistent variables.
Sorry, something went wrong.
powerlevel is correct, it's one word in all Z-Wave docs.
@kpine Yes, typo! Thanks
5b3a205
No branches or pull requests
I think this should be
powerlevel
, notpowerLevel
?zwave-js-ui/src/stores/base.js
Line 28 in 1741fac
Referenced here (I think):
zwave-js-ui/lib/ZwaveClient.ts
Lines 2137 to 2143 in 1741fac
I don't think it's causing an issue if you change settings via the UI, but something I noticed.
The text was updated successfully, but these errors were encountered: