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

[waybar] Weather custom module isn't working #24

Closed
vnourdin opened this issue Jan 28, 2022 · 8 comments
Closed

[waybar] Weather custom module isn't working #24

vnourdin opened this issue Jan 28, 2022 · 8 comments

Comments

@vnourdin
Copy link
Contributor

The weather custom Waybar module seems broken, as spotted in #22.

I don't entirely understand how Waybar's modules works, but the weather.sh script takes argument but I don't see any arg passed from the Waybar config.

@OdiousImp2604
Copy link
Collaborator

Yep. Commented it out for now. Been a little busy so haven't found an alternative.

@OdiousImp2604
Copy link
Collaborator

I was thinking of replacing the weather script with with this. https://gist.github.com/sveatlo/d2fb837758088a98ad727f56072d6f1c. excluding the geolaction API.

All credit to svealto

@ringo32
Copy link
Contributor

ringo32 commented Aug 7, 2022

Could use wttr.in?

@OdiousImp2604
Copy link
Collaborator

Could use wttr.in?

Oo good idea I will look at that.

@ringo32
Copy link
Contributor

ringo32 commented Aug 7, 2022

in my lxqt panel is like : echo $(curl https://wttr.in/Pelt:Lommel:Hamont?format=4 )
with this : echo $(curl https://wttr.in/?format=4 ) wil it choose the region not the exact place , could be a village beside :)

the 4 means also get symbols of wint sunny night etc..

only bug it wil be also, sometimes are the servers overload some days then it get a error on the panel. 1M calls a day i believe.

@ghost
Copy link

ghost commented Aug 9, 2022

I made a pull request to fix this issue but it got rejected without clarification

@OdiousImp2604
Copy link
Collaborator

I made a pull request to fix this issue but it got rejected without clarification

I'm unsure who actually closed that. Sorry, let me take another look right now.

@OdiousImp2604
Copy link
Collaborator

I made a pull request to fix this issue but it got rejected without clarification

Now merged closing issue

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

No branches or pull requests

3 participants