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

weather forecast #21

Open
fabricocouto opened this issue Aug 29, 2022 · 7 comments · Fixed by #25
Open

weather forecast #21

fabricocouto opened this issue Aug 29, 2022 · 7 comments · Fixed by #25
Assignees

Comments

@fabricocouto
Copy link

also the weather forecast system would consider changing to another api platform, or that no longer works it would be nice to put the weather on top of the avatar

@Swizzy
Copy link
Contributor

Swizzy commented Aug 30, 2022

Feel free to update it with support for a different API if you want - adding on top of the avatar i'm not sure what you mean by that, try showing with a screenshot perhaps?

@fabricocouto
Copy link
Author

Unfortunately, I don't understand much about lua writing, so connoisseur of C#,
but i can help with ideas

@fabricocouto
Copy link
Author

imagem_2022-08-30_121435970
I thought of it for the weather forecast

@Swizzy
Copy link
Contributor

Swizzy commented Aug 30, 2022

I don't think there currently is support for that level of plugins in Aurora unfortunately. I also think this should probably be more of a core feature to be displayed like this.

It's a nice idea, don't get me wrong - but, it'll likely take some time before this could become reality, if it ever comes since like i said it should be part of the core and not a plugin per say.

@Ste1io Ste1io self-assigned this Jul 16, 2024
@Ste1io
Copy link
Contributor

Ste1io commented Jul 19, 2024

I've updated the script to use Open-Meteo API instead of the previous now-defunct service, and improved the "onboarding" location search feature to accept city, postal code, or lat,lon. Will push changes tonight after adding a basic validation check for the settings.ini.

Bonus: no longer requires an api key; just set location and you're all set.

@fabricocouto I like the presentation in your mockup, like a weather widget essentially; like @Swizzy said, that would require some changes to core. I'll keep it in mind though.

1721356243-192 168 137 99-image127 bmp_‎-_Photos

1721356259-192 168 137 99-image128 bmp_‎-_Photos

1721354637-192 168 137 99-image125 bmp_‎-_Photos

1721354645-192 168 137 99-image126 bmp_‎-_Photos

@fabricocouto
Copy link
Author

It was great, congratulations on the work

@Ste1io
Copy link
Contributor

Ste1io commented Aug 25, 2024

been meaning to push a final commit with the remaining fixes; will do that this weekend so we can close this issue out.

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 a pull request may close this issue.

3 participants