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

[vesync] Add auto-mode to core 300S devices #17842

Merged
merged 4 commits into from
Dec 5, 2024

Conversation

dag81
Copy link
Contributor

@dag81 dag81 commented Dec 3, 2024

[vesync] Core 300S Updates

This updates the device profile for the core 300S. This adds the auto mode to the fan modes for its profile. It looks like since the initial ports the modes specified for the device have been updated.

This change adds the auto mode as a valid payload for the protocol to allow for this device.

This relates to OpenHab issue: [#16456] (#16456).

The changes have been confirmed as working as expected.

dag81 and others added 4 commits December 3, 2024 23:45
[vesync] Core 300S Updates

Signed-off-by: David Goodyear <david.goodyear@gmail.com>
[vesync] Cleanup

Signed-off-by: David Goodyear <david.goodyear@gmail.com>
[vesync] Remove fan speed adjust

Signed-off-by: David Goodyear <david.goodyear@gmail.com>
@dag81 dag81 marked this pull request as ready for review December 5, 2024 00:45
@lsiepel lsiepel added the enhancement An enhancement or new feature for an existing add-on label Dec 5, 2024
@lsiepel lsiepel self-requested a review December 5, 2024 19:17
@lsiepel lsiepel linked an issue Dec 5, 2024 that may be closed by this pull request
Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@lsiepel lsiepel merged commit ca2ee20 into openhab:main Dec 5, 2024
5 checks passed
@lsiepel lsiepel added this to the 4.3 milestone Dec 5, 2024
@lsiepel lsiepel changed the title [vesync] Core 300S Updates [vesync] Add auto-mode to core 300S devices Dec 5, 2024
@dag81
Copy link
Contributor Author

dag81 commented Dec 5, 2024

Thank you for the quick merge @lsiepel! 👍

@dag81 dag81 deleted the Core300SUpdates branch December 5, 2024 23:57
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Dec 16, 2024
* [vesync] Core 300S Updates

Signed-off-by: David Goodyear <david.goodyear@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[VeSync] Add "Change mode to auto" for Air Purifier Core 300S
2 participants