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

Feature spa heater functionality #8

Open
wants to merge 25 commits into
base: main
Choose a base branch
from

Conversation

cupshir
Copy link
Owner

@cupshir cupshir commented Apr 29, 2023

Refactor to support new functionality for toggling the pool, spa, or spillover mode and adding heater accessory.

homebridge-fun and others added 25 commits April 9, 2023 12:29
this change is specific to build errors
had to make changes to noImplicitAny due to the following error during build.  

error TS7016: Could not find a declaration file for module '@homebridge/dbus-native'
had to make changes to noImplicitAny due to the following error during build.  

error TS7016: Could not find a declaration file for module '@homebridge/dbus-native'
-new functionality to parse pool mode
-changed GetDeviceState to ParseDeviceState
-moved api calls out of parse functions into its own function getResponse
-dded new sleep function
-added requester parameters to functions for better logging
-moved checking device state into its own function so it can be used by both setOn and getOn
-updated setOn and getOn to use the new ParseState
-refactored setOn for better efficiency and readability
-tweaked logging messages to be a bit more readable
initial commit of new device used for toggling pool, spa, and or spillover.
-added missing log entry for when getResponse fails.
-refactored to use named exports
-added ACCESSORY_MODE const
-renamed SpaMode to Spa
-added Pool and Spillover accessories
-
-refactored how accessories are loaded
-added functionality for new ModeSwitch device, for Pool, Spa, & Spillover
-added cacheing functionality for api requests made to the AquaConnect bridge
-added check to ensure disclaimer has been accepted
-disabled noImplicitAny
-added new disclaimer
-updated all settings to be hidden until disclaimer is accepted
-added new setttings for
--Get Request Cache Threshold
--Get Request Delay
--Set Request Delay
-Added Pool, Spa, Spillover, and Heater to Include Accessories
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

Successfully merging this pull request may close these issues.

2 participants