Change from Fahrenheit to Celsius about angular-sh-weather HOT 6 CLOSED

Hi Winsehr,

Just wanted to let you know I'm taking a look at adding this in it should be a pretty simple change but the component has been neglected so I'm needing to fix dependencies before I can test things and add this as an option.

Thanks

from angular-sh-weather.

Alright updated the code, if you bump from v0.0.0 to v0.0.1 you can now use units="metric" in the directive usage to show degrees celsius instead of fahrenheit.

Note since I had to fix up the build process the final locations for the minified file moved so beware you'll need to update your script tag accordingly.

from angular-sh-weather.

Had a problem here when I released the v0.0.1 tag so just skip that and go with v0.0.2

from angular-sh-weather.

Hi, again!
Thank you so much for the update but now I get an error in console when I try to use the widget. It says "Error: [$compile:tplrt] Template for directive 'todaysWeather' must have exactly one root element. templates/currentWeatherDisplay.tpl.html". I've tried to go back to previous versions but I get the same error.

from angular-sh-weather.

Strange I didn't change too much to do with the template, will try to test this out in a clean project in the next few days to see what happened, might just be some bad commit was having some troubles with the order I updated the version numbers and added the tag.

Make sure you use v0.0.2 as well since I know for sure v0.0.1 was a broken build.

from angular-sh-weather.

Hello!
It's OK, I can wait so you take your time :)
Yes, I used 0.0.2!

from angular-sh-weather.