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

Added thermostatTemperatureSetpointHigh and ...Low to Metadadata UI list #721

Merged
merged 2 commits into from
Dec 29, 2020
Merged

Added thermostatTemperatureSetpointHigh and ...Low to Metadadata UI list #721

merged 2 commits into from
Dec 29, 2020

Conversation

eikowagenknecht
Copy link
Contributor

@eikowagenknecht eikowagenknecht commented Dec 28, 2020

Fixes various problems with the google assistant metadata UI, including #720. I took all the data directly from the source, not relying on the documentation (because it is wrong in some minor places).

  • Add thermostatTemperatureSetpointHigh and thermostatTemperatureSetpointLow (fixes Google Assistant metadata missing optional thermostat items #720) image
  • Rename CoffeeMaker to Coffee_Maker (yes, it has to be this inconsistent to work unfortunately, thanks Google...) image
  • Add protocols and tokens parameters for Camera image
  • Rename wrongly named parameter speed to speeds and add parameter lang (Fan)
    image
  • Add advanced parameter structureHint (all items) because it is implemented in the assistant binding and roomHint is also in the UI already
    image
  • Add parameter inverted for Switch, Outlet, CoffeeMaker, WaterHeater, Fireplace, Valve, Sprinkler, Vacuum, Lock, SecuritySystem
    image
  • Add parameter thermostatTemperatureRange (Thermostat)
    image
  • Added some examples in the descriptions of the parameters (see screenshots above)

Signed-off-by: Eiko Wagenknecht eiko.wagenknecht@web.de

…ta UI list

Signed-off-by: Eiko Wagenknecht <eiko.wagenknecht@web.de>
@eikowagenknecht eikowagenknecht requested a review from a team as a code owner December 28, 2020 18:25
Signed-off-by: Eiko Wagenknecht <eiko.wagenknecht@web.de>
@eikowagenknecht
Copy link
Contributor Author

Just a side note: The coffee maker issue is also documented here: openhab/openhab-google-assistant#154

Copy link
Member

@ghys ghys left a comment

Choose a reason for hiding this comment

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

Awesome, I don't use the Google assistant so I made this editor from the docs and it's largely untested, it's great that you're able to make it work in practice!

@ghys ghys merged commit 77106ab into openhab:master Dec 29, 2020
@eikowagenknecht eikowagenknecht deleted the fix-720 branch December 29, 2020 15:21
crnjan added a commit to crnjan/openhab-webui that referenced this pull request Jan 6, 2021
* master:
  Take all door subclasses in door location glance badge (openhab#727)
  [habpanel] Remove ItemStateEvent detection (openhab#737)
  Update license headers to 2021 (openhab#739)
  Fix openhab#714 - Sort popup dialogs by name and jump to selected item on open (openhab#724)
  Fix openhab#538 - Do not concatenate search for items (openhab#726)
  Fix openhab#722 - Show current GA metadata when editing (openhab#723)
  Fix openhab#720 - Fix various problems with google assistant metadata UI (openhab#721)
  Fix openhab#684 - Allow diacritics in search bars (openhab#718)
  CSP: allow blob: URLs (openhab#698)
  Apply Spotless (openhab#680)
  [unleash-maven-plugin] Preparation for next development cycle.
@ghys ghys added the bug Something isn't working label Jan 11, 2021
@ghys ghys added this to the 3.1 milestone Jan 11, 2021
ghys pushed a commit to ghys/openhab-webui that referenced this pull request Jan 11, 2021
… UI (openhab#721)

Signed-off-by: Eiko Wagenknecht <eiko.wagenknecht@web.de>
@ghys ghys added patch A PR that has been cherry-picked to a patch release branch main ui Main UI labels Jan 11, 2021
@kaikreuzer kaikreuzer changed the title Fix #720 - Add thermostatTemperatureSetpointHigh and ...Low to Metadadata UI list Added thermostatTemperatureSetpointHigh and ...Low to Metadadata UI list Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working main ui Main UI patch A PR that has been cherry-picked to a patch release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Google Assistant metadata missing optional thermostat items
2 participants