-
-
Notifications
You must be signed in to change notification settings - Fork 46
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
Allow alternative services to be selected #458
Conversation
Codecov Report
@@ Coverage Diff @@
## master #458 +/- ##
==========================================
+ Coverage 50.36% 53.30% +2.94%
==========================================
Files 23 36 +13
Lines 1938 2088 +150
Branches 509 534 +25
==========================================
+ Hits 976 1113 +137
+ Misses 509 507 -2
- Partials 453 468 +15
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
…Motion Sensor instead of Occupancy Sensor.
🆗 Published SonarCloud and code coverage data. |
Kudos, SonarCloud Quality Gate passed! |
Trying out some things on this branch to allow for configuration that is specific to a converter, which would allow users to configure certain converters to behave differently (i.e. expose a different services, like an Outlet instead of a Switch).
Hopefully in the end this will provide a solution for #45, #279, #380, #422 and possibly other issues/requests I'm forgetting here.
No promises yet, as I still feel this approach requires some polishing.
Definitely still to do:
basic_sensors.ts
(information about the properties should be in the class that handles it rather than in the "creator collection")