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

Update weather.js #32

Merged
merged 1 commit into from
Nov 1, 2015
Merged

Update weather.js #32

merged 1 commit into from
Nov 1, 2015

Conversation

LithiumNL
Copy link
Contributor

The sunset icon was never displayed because the if statement was different. The sunset and sunrise were are in HH:MM and now in another timestamp.
_sunset > now should be _sunset > _now

The sunset icon was never displayed because the if statement was different. The sunset and sunrise were are in HH:MM and now in another timestamp.
_sunset > now should be _sunset > _now
@LithiumNL LithiumNL closed this Nov 1, 2015
@LithiumNL LithiumNL reopened this Nov 1, 2015
MichMich added a commit that referenced this pull request Nov 1, 2015
@MichMich MichMich merged commit 067ef7e into MagicMirrorOrg:master Nov 1, 2015
@MichMich
Copy link
Collaborator

MichMich commented Nov 1, 2015

Cool, Thanks!

@LithiumNL
Copy link
Contributor Author

LithiumNL commented Nov 1, 2015 via email

Broutard pushed a commit to Broutard/MagicMirror that referenced this pull request Jan 9, 2016
MichMich pushed a commit that referenced this pull request Jan 12, 2020
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