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

pyscript compatibility #2980

Open
1 task done
dirtyharryiv opened this issue Sep 29, 2024 · 3 comments
Open
1 task done

pyscript compatibility #2980

dirtyharryiv opened this issue Sep 29, 2024 · 3 comments
Labels

Comments

@dirtyharryiv
Copy link

What version of Music Assistant has the issue?

2.2.6

What version of the Home Assistant Integration have you got installed?

2024.9.1

Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

  • Yes

The problem

I can't call the mass.search service via pyscript. When i try to call the service with minimal params i get the folowing error:
TypeError: Function.service_call() got multiple values for argument 'name'

If somebody could change line 23 in services.py from
ATTR_SEARCH_NAME = "name"
to
ATTR_SEARCH_NAME = "searchname"
i'm pretty sure it would work again.

How to reproduce

In pyscript do:
service.call("mass", "search", name="queen")

Music Providers

youtube music

Player Providers

chromecast

Full log output

No response

Additional information

No response

What version of Home Assistant Core are your running

2024.9.3

What type of installation are you running?

Home Assistant OS

On what type of hardware are you running?

Generic x86-64 (e.g. Intel NUC)

@OzGav
Copy link
Contributor

OzGav commented Oct 15, 2024

You can mess around with the custom component directly. So you can change that line 23 and reboot HA and see how you go.

@dirtyharryiv
Copy link
Author

Thanks for your answer! I did what you suggested before opening the issue and it works. But would be nice if the next update wouldn't break it.

@OzGav
Copy link
Contributor

OzGav commented Oct 15, 2024

You can open a PR and see what the devs think

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants