Input...What do you think?

cowboysdude

@Snille Thank you I changed it and THANK YOU FOR NOTICING… I also discovered that I left my api key in the main.js file…ughhhhhhhhhhh had to change that too. I know I was tired when I posted this on github… hahahahhaah

cowboysdude

@Snille What did you use for location?

Like I use NY/Elmira

We have to see if your location is working…

cowboysdude

@Snille said in Input...What do you think?:

@cowboysdude Hmm… I forgot a “comma” on the
en: "translations/en.json"
line… Sorry…
should be:
en: "translations/en.json",
And then it seems to be missing something else…
I get this error…
Uncaught TypeError: Cannot read property 'simpleforecast' of undefined
    at Class.processNoaa (MMM-NOAA.js:58)
    at Class.socketNotificationReceived (MMM-NOAA.js:88)...
But I have to go to bed… REALLY now… :)

It doesn’t recognize your location :(

Mykle1

@Snille

“250,000+ personal weather stations across the globe already send data to Weather Underground!”

Not a single one in Sweden, according to https://www.wunderground.com/weatherstation/ListStations.asp?selectedCountry=Sweden

cowboysdude

@Mykle1 said in Input...What do you think?:

@Snille

“250,000+ personal weather stations across the globe already send data to Weather Underground!”

Not a single one in Sweden, according to https://www.wunderground.com/weatherstation/ListStations.asp?selectedCountry=Sweden

That’s NO GOOD.

Mykle1

@cowboysdude

@Snille’s next project? :-)

cowboysdude

@Mykle1 OH wait don’t I have like 2 other weather modules I made and NEVER released? Perhaps I can salvage one of those… hummmmmmmmmmmmm

Mykle1

@cowboysdude

We don’t want the other ones. We want this one!

Make it work! :-)

cowboysdude

Thanks to Mykle we’ve changed the api call and NOW it should cover just about the entire world … well if you have a pws close by you :)

Thanks to @Snille and @Mykle1 for their help… like I said didn’t plan on releasing this and honestly only asking for what others thought of it but hey looks like once the bugs are all ironed out it’ll be coming to a mirror near you…

barnosch

Looks very nice.
The german translation seems to be wrong ;) and not even added to the MMM-NOAA.js
Guess your are still in the middle of development.
Added it myself but it is not working yet.
Also i would like a 24hours setting, if possible.

Thansk for your work

PS: is this correct in the default settings? Guess not.

show: "F".toLowerCase(), // show F or C Temps