Skip to content

Conversation

@SkySails
Copy link
Contributor

This PR does not close any issues. It does, however, aim to improve the SMHI weather provider with support for hourly forecasts and apparent ("Feels like") temperature, along with the ability to customize the name of the location.

@codecov-commenter
Copy link

codecov-commenter commented Aug 30, 2022

Codecov Report

Merging #2902 (50f72f0) into develop (ba128cb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2902   +/-   ##
========================================
  Coverage    63.82%   63.82%           
========================================
  Files            9        9           
  Lines          293      293           
========================================
  Hits           187      187           
  Misses         106      106           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@MichMich
Copy link
Collaborator

modules/default/weather/providers/smhi.js [53](https://github.com/MichMich/MagicMirror/runs/8085290605?check_suite_focus=true#step:4:54) [warn] Code style issues found in the above file. Forgot to run Prettier?

Could you fix the issue above? Thanks!

@SkySails
Copy link
Contributor Author

@MichMich Of course! Sorry about that.

@MichMich MichMich merged commit ca906c4 into MagicMirrorOrg:develop Aug 30, 2022
@MichMich
Copy link
Collaborator

No worries! Thanks! Merged.

MichMich pushed a commit that referenced this pull request Oct 4, 2022
This PR addresses [this
comment](48756e8#commitcomment-85772193),
which points out an issue with #2902.

Looks like the apparent temp calculation method was incorrectly
referenced 😅
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.

5 participants