-
Couldn't load subscription status.
- Fork 514
WWSTCERT-8423/8426 [Aqara] Wireless Remote Switch T1(Single/Double) #2458
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
base: main
Are you sure you want to change the base?
WWSTCERT-8423/8426 [Aqara] Wireless Remote Switch T1(Single/Double) #2458
Conversation
|
Duplicate profile check: Passed - no duplicate profiles detected. |
|
Invitation URL: |
Test Results 71 files 459 suites 0s ⏱️ Results for commit 9155130. ♻️ This comment has been updated with latest results. |
|
Minimum allowed coverage is Generated by 🐒 cobertura-action against 9155130 |
| - id: refresh | ||
| version: 1 | ||
| categories: | ||
| - name: Button |
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.
@seojune79 Could you change this to RemoteController also?
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.
@Kwang-Hui Yes, I've reflected that.
…ontroller' in the 'aqara-double-buttons' profile
…named the file to one-button-batteryLevel.yml.
| [1] = { | ||
| id = 1, | ||
| manufacturer = "LUMI", | ||
| model = "lumi.remote.acn003", |
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 device should still be using the one-button-battery profile, as you have not updated the fingerprint.
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.
I have updated the fingerprints for all Aqara Button Devices by implementing the batteryLevel capability.
| - id: refresh | ||
| version: 1 |
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 profile needs to include firmwareUpdate
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.
Okay, I have added the firmwareUpdate capability.
Check all that apply
Type of Change
Checklist
Description of Change
Summary of Completed Tests