Changing units from 'metric' to 'imperial' breaks currentweather module
Changing my config.js units from ‘metric’ to ‘imperial’ causes my currentweather module not to appear. Any thoughts?
@zorrotech seems I’m having the same issue, I just posted about it not working, but I now see that I too am having this same issue. The weatherforcast module still seems to work in imperial but the current weather did break.
So glad to know that someone else is experiencing the same thing (well, not actually glad that someone else is having a problem…) All was well until performing the v2.3.1 update, yesterday. The weatherforecast module is fine, but the current weather module is not displaying, and I do have the imperial unit option set.
Confirmed that switching to metric restores a display.
@dplawrance its fixed on the develop branch
Agreed it does appear to fix what was broken. :)
Wow I was really starting to lose my mind over this one. Have been going over and over my config.js file.
Having this issue as well, with the same reproduction, changing to metric brings it back but imperial breaks it.
How do I pull the file from the develop branch, as someone fairly new to all this?
You might even be able to use
git cherry-pick 10eb41d31943a4495bf4c4674d52d0b9631a7446instead, to stay on
masterbut incorporate that specific fix. YMMV.
@ninjabreadman maybe a silly question, but can I… download and manually replace the file? git cherry-pick failed with the reason:
error: cherry-pick is not possible because you have unmerged files.
hint: Fix them up in the work tree, and then use 'git add/rm '
hint: as appropriate to mark resolution and make a commit, or use
hint: ‘git commit -a’.
fatal: cherry-pick failed
edit… scratch that, it just started working again…