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

feat: rebuild device list #7

Closed
wants to merge 101 commits into from

Conversation

mikejgray
Copy link

NeonDaniel and others added 30 commits March 8, 2023 15:56
Add stable version specs to all dependencies
fix: repo URL in readme was AIIX's original
* feat: add support for Scenes

* chore: cleanup build/dist and add gitignore

* chore: add missing requirement

* feat: automation support

* feat: add gui cards for scenes and automations

* add switch card back

* chore: deal with git issues

* fix naming

* feat: handle passthrough of Assist messages

This will allow us to keep up with all features Home Assistant Assist offers without explicitly implementing them.

Tests forthcoming.

* chore: correct name of event

* chore: add debug log

* fix: disable turning off automations

This disables the automation, which is definitely not what we want.

* corrections to method + black formatting

* adjust behavior of automation card

"Turn off" disables the automation, so we only ever want this to "turn on" the automation (trigger it).

* chore: add icons for new cards

* fix: disable turning off automations

This disables the automation, which is definitely not what we want.

* adjust behavior of automation card

"Turn off" disables the automation, so we only ever want this to "turn on" the automation (trigger it).

* chore: add icons for new cards

* chore: align debug config with standard

* remove additional layer of DEBUG logging

* attach log to plugin instance

* remove self.log

* force log level

* feat: add support for Scenes

* chore: cleanup build/dist and add gitignore

* chore: add missing requirement

* feat: automation support

* feat: add gui cards for scenes and automations

* chore: deal with git issues

* fix naming

* feat: handle passthrough of Assist messages

This will allow us to keep up with all features Home Assistant Assist offers without explicitly implementing them.

Tests forthcoming.

* chore: correct name of event

* chore: add debug log

* fix: disable turning off automations

This disables the automation, which is definitely not what we want.

* corrections to method + black formatting

* adjust behavior of automation card

"Turn off" disables the automation, so we only ever want this to "turn on" the automation (trigger it).

* chore: add icons for new cards

* chore: align debug config with standard

* remove additional layer of DEBUG logging

* attach log to plugin instance

* remove self.log

* force log level

* fix: match the Message spec

data should never be a list, only a dict or NoneType

* refactor: bring in utils from vui


refactor: get single device plus tests


fix typo

add forced return

another typo fix

turn devices on and off

also improves tests for getting device info

better response for vui

type correction

try to handle mutations

add tests

* python not js

* remove unsupported type in python 3.7

* refactor call_supported_function and handle response

* handle requests for light brightness

adds methods for increasing/decreasing brightness by 10%

* add handler for decreasing light brightness

* add logging

* correct function

* correct function

* math

* update device state after changes

* get light color

* tests and fixes from tests

* method fix

* set light color

* defensive programming

* fix function call

* adjustment to installation steps

* fix broken scene card

* adjust text

* review feedback
* Remove leftover `mycroft-messagebus-client` dependency replaced by `ovos-bus-client`

* Update OPM dependency spec

* Fix leftover `mycroft-bus-client` impmort
Reformat file for PEP
@mikejgray mikejgray closed this Jan 24, 2025
@mikejgray mikejgray deleted the FEAT_RebuildDeviceList branch January 25, 2025 16:59
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.

5 participants