crowbarz / aiopioneer

Pioneer AVR API (asyncio)
Apache License 2.0
8 stars 6 forks source link

Add set_source_name function #23

Closed pantherale0 closed 1 year ago

pantherale0 commented 1 year ago

Adds a basic function to set and reset source names. (replicating iControlAV5 functionality)

Source names cannot be longer than 14 characters so a ValueError is raised in the event that a new name is provided longer than 14 chars.

Also, setting the default paramter to True will reset the source name to its default value.

crowbarz commented 1 year ago

Hmmm, noticed on writing release notes for the HA integration that set_source_name is actually called set_input_name. Thinking that inputs are referred to everywhere else in the HA integration and aiopioneer as source, so will rename the function and commands before adding on the HA side.

pantherale0 commented 1 year ago

Hmm, I actually thought I changed that, I remember I started with set_input_name but changed it to set_source_name. Must have forgotten to commit that change before rebasing the branch to reflect the later changes.

pantherale0 commented 1 year ago

Ohh, I see, I changed it in https://github.com/crowbarz/aiopioneer/pull/23/commits/caaedb689d0606ef67eb4c92247ee22bedd4a200