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

Adding Open Weather support #51

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

Adding Open Weather support #51

wants to merge 3 commits into from

Conversation

jakezp
Copy link

@jakezp jakezp commented Oct 12, 2020

No description provided.

@JWDebler
Copy link

JWDebler commented May 2, 2023

Hi @jakezp,

I've added your changes to my local install and signed up at openweathermap.org.

I keep getthing these errors though:
WARNING 401 Client Error: Unauthorized for url: https://api.openweathermap.org/data/2.5/onecall?lat=XXX&lon=XXX&units=metric&exclude=minutely,hourly,daily,alerts&appid=XXX

On https://openweathermap.org/appid they have a slightly different way of calling the api:
http://api.openweathermap.org/data/2.5/forecast?id=524901&appid=[{API key}]

Do I have to sign up for one of their paid API services, or should this work with the free one that comes automatically after signing up?

Cheers

@JWDebler
Copy link

JWDebler commented May 2, 2023

Got it, signed up for the free OneCall 3.0 API. Took a few minutes until it was activated and I stopped getting the error.

Also, the ow_api.py needs to be updated to reflect the new API version from:

https://api.openweathermap.org/data/2.5/onecall?...
to
https://api.openweathermap.org/data/3.0/onecall?...

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.

2 participants