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

Update audi_services.py #203

Merged
merged 1 commit into from
Feb 1, 2024
Merged

Update audi_services.py #203

merged 1 commit into from
Feb 1, 2024

Conversation

reisfni
Copy link
Contributor

@reisfni reisfni commented Oct 3, 2023

Fixes issue in Home Assistant where device/entities were not populating after adding the integration.

Fixed HASS issues
Copy link

@damienheiser damienheiser left a comment

Choose a reason for hiding this comment

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

I've made the change to my HA and can confirm it works as desired.

@florie1706
Copy link

florie1706 commented Oct 11, 2023

I am in Germany. Old Url still worked, I changed it to the Url of this pull request anyway, but that did not work for me. So I am on the old Url again which still works for me.

@agentadam-code
Copy link

WOW this works. Thanks so much.

@oleg-d
Copy link

oleg-d commented Nov 11, 2023

I am in Germany. Old Url still worked, I changed it to the Url of this pull request anyway, but that did not work for me. So I am on the old Url again which still works for me.

Same for me, old URL was working and new one is not. I'm in Finland.

@woopstar
Copy link
Contributor

I am in Germany. Old Url still worked, I changed it to the Url of this pull request anyway, but that did not work for me. So I am on the old Url again which still works for me.

Same for me, old URL was working and new one is not. I'm in Finland.

Same for me in Denmark. Old url works, not new one.

@victorkp
Copy link
Contributor

US user here -- this updated URL works.

We should be switching URLs dependent the user's specified region.

@victorkp
Copy link
Contributor

I've made pull request #211, which only sets the new URL for US users. Non-US users will continue to use the old URL. @reisfni, you should close this PR.

@arjenvrh arjenvrh merged commit 2f34574 into audiconnect:master Feb 1, 2024
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.

8 participants