Skip to content
This repository has been archived by the owner on Apr 26, 2022. It is now read-only.

ANNOUNCE_WEATHER_HOURS should also include minutes #6

Open
rlueder opened this issue Jan 21, 2022 · 0 comments
Open

ANNOUNCE_WEATHER_HOURS should also include minutes #6

rlueder opened this issue Jan 21, 2022 · 0 comments
Assignees
@rlueder
Labels
bug Something isn't working
Milestone
1.1.0

Comments

@rlueder
Copy link
Owner

rlueder commented Jan 21, 2022 •
edited
Loading

As it stands ANNOUNCE_WEATHER_HOURS will announce the weather multiple times within the hour. The array should perhaps use strings for full time e.g. [ "00:00", "06:00", "12:00", "18:00"]

This should be enough to make it so that weather forecasts are announced only once within the intervals.
@rlueder rlueder added the bug Something isn't working label Jan 21, 2022
@rlueder rlueder self-assigned this Jan 21, 2022
@rlueder rlueder added this to the 1.1.0 milestone Jan 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rlueder rlueder

Labels
bug Something isn't working
Projects
Spicetify Announcer
Status: In Progress
Milestone
1.1.0
Development

No branches or pull requests
1 participant
@rlueder