Skip to content

Conversation

@legojrp
Copy link

@legojrp legojrp commented Apr 8, 2025

Added functionality of weather to be displayed in the header.
It requires location services, user will be prompted for their location. It also requires an API Key from OpenWeatherMap.org, which is free as long as someone doesn't open their notch 60 times per minute. (Edit: previous stat was wrong)

Showcase:

screen-recording-2025-04-08-at-170010_SK18rw3J.mp4

It's off by default, and won't request your location unless you specifically go in settings to turn it on.

Disclaimer: I did use some AI on this. I'm not a great SwiftUI person, so if there is anything wrong or bad syntax or whatever, my apologies.

@Alexander5015
Copy link
Member

Thanks for the contribution! I will review it fully after the next release. One thing I noticed when briefly skimming it is that you accidentally added entitlements for Apple Music and Spotify. Not a big deal, but you should remove those so it doesn't ask for Spotify permission before the user selects Spotify in settings.

A few notes for features that should be included(I don't know if these exist or not):

  • Ability to use a fixed location rather than location services
  • Save results and only update if the last update was more than 5 or 10 minutes ago, or the location has changed
  • Get an updated location using a fixed rate as well

@Alexander5015 Alexander5015 deleted the branch TheBoredTeam:dev August 2, 2025 06:21
@Alexander5015 Alexander5015 reopened this Aug 2, 2025
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