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

Change url to HA-dedicated url #36

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Olen
Copy link
Contributor

@Olen Olen commented Feb 9, 2024

Should fix #34 and #35

@Olen
Copy link
Contributor Author

Olen commented Feb 9, 2024

Somethings not right.... Will be back:

Retrying in 15 minutes: not well-formed (invalid token): line 1, column 0

@Olen Olen marked this pull request as draft February 9, 2024 13:44
@Olen
Copy link
Contributor Author

Olen commented Feb 9, 2024

Hm.
The "new" API returns json instead of XML...
And the parameter names have changed a lot.
This seems to work locally, but probably needs some improvements.

@Olen Olen marked this pull request as ready for review February 9, 2024 14:32
@ways
Copy link

ways commented Feb 12, 2024

Hm. The "new" API returns json instead of XML... And the parameter names have changed a lot. This seems to work locally, but probably needs some improvements.

There is no new API. You can choose xml or json. Check the api docs.

@Dehumanizer77
Copy link

I can confirm this works for me as well.

@gpongelli
Copy link
Contributor

I confirm those changes works like a charm 😃

@jeppper
Copy link

jeppper commented Mar 9, 2024

What changes needs to be changed where?

@jeppper jeppper mentioned this pull request Apr 19, 2024
@jeppper
Copy link

jeppper commented Sep 29, 2024

What changes needs to be changed where?

@gpongelli
Copy link
Contributor

hi @Danielhiversen this PR fixes the integration, please merge it and release a new version, thanks 🙏

Copy link
Owner

@Danielhiversen Danielhiversen left a comment

Choose a reason for hiding this comment

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

https://aa015h6buqvih86i1.api.met

Should only be used by the official HA integration

@jeppper
Copy link

jeppper commented Jan 18, 2025

Would it be possible to make this integration work with an old version of HA 0.103.6?

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.

Blocked by Met?
6 participants