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

Theme Patcher Bug? #79

Open
3 tasks done
Mar1usW3 opened this issue Feb 3, 2025 · 4 comments
Open
3 tasks done

Theme Patcher Bug? #79

Mar1usW3 opened this issue Feb 3, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@Mar1usW3
Copy link

Mar1usW3 commented Feb 3, 2025

Bug Report

If i try to execute this with the theme patcher in Homeassistant:

  - action: shell_command.create_token
    data:
      theme: graphite
      token: brown-color
      type: generic
      value: rgb(var(--token-rgb-red))

i get the following error:

Update & Patch Graphite Theme (Advanced): Error executing script. Service not found for call_service at pos 4: Action shell_command.create_token not found

Then i execute this, the log say it is done but nothing is done :-)

action: shell_command.patch_theme
data:
  theme: graphite
  token: red-color
  type: generic
  value: rgb(var(--token-rgb-green))

if i execute:

action: shell_command.patch_theme
data:
  theme: graphite
  token: 123-color
  type: generic
  value: rgb(var(--token-rgb-green))
this is working

Quick Checklist

  • I am using the latest version of Home Assistant.
  • I cleared my browser cache.
  • I checked the documentation and existing issues.
@Mar1usW3 Mar1usW3 added the bug Something isn't working label Feb 3, 2025
@TilmanGriesel
Copy link
Owner

@Mar1usW3 I’m trying to figure this out, but it’s unclear what’s causing the issue. It looks like you’re calling a different shell command named create_token, which might be missing its configuration. Could it be that you need to add the token creation flag to shell_command.patch_theme and use that instead?

@Mar1usW3
Copy link
Author

i saw this here:
#76 (comment)
but i tought this is not needed anymore

@TilmanGriesel
Copy link
Owner

The flag is still required, but I have already included it in the Advanced Shell Command Example, so no additional modifications are needed.

If possible, I recommend using a single shell command entry. Just ensure you have the latest shell command example and the most up-to-date theme patcher script. You can update to the latest version by running: Step 1 – Download the Patcher.

Managing multiple scripts and methods can quickly become convoluted and confusing. Hope that helps!

@Mar1usW3
Copy link
Author

i only use one single shell command:

patch_theme: "python3 /config/scripts/graphite-theme-patcher.py --theme '{{ theme }}' --token '{{ token }}' --type '{{ type }}' --create --value '{{ value }}'"

but still the red-color token is still not created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants