-
Notifications
You must be signed in to change notification settings - Fork 466
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
[Aqara] Humidity_temp_sensor Battery capability change #1853
base: main
Are you sure you want to change the base?
[Aqara] Humidity_temp_sensor Battery capability change #1853
Conversation
end | ||
device:emit_event(capabilities.batteryLevel.battery(batteryLevel)) | ||
end | ||
|
||
local function device_init(driver, device) | ||
battery_defaults.build_linear_voltage_init(2.6, 3.0)(driver, device) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be removed
|
Hi @lelandblue , I couldn't find driver sharing link for this PR. could you check? |
Duplicate profile check: Passed - no duplicate profiles detected. |
Invitation URL: |
@@ -53,27 +53,49 @@ local is_aqara_products = function(opts, driver, device) | |||
return false | |||
end | |||
|
|||
local function device_init(driver, device) | |||
battery_defaults.build_linear_voltage_init(2.6, 3.0)(driver, device) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
battery_defaults
is now unused
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is unused and replaced with battery_level_handler
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right. But the require
needs to be removed. That's why luacheck failed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh. I get it. @hjl-aqara , Could you remove line number 1?
- local battery_defaults = require "st.zigbee.defaults.battery_defaults"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I just took care of it.
Test Results 64 files 404 suites 0s ⏱️ Results for commit 7775fb9. ♻️ This comment has been updated with latest results. |
Minimum allowed coverage is Generated by 🐒 cobertura-action against 7775fb9 |
Check all that apply
Type of Change
Checklist
Description of Change
Summary of Completed Tests