MMM-NOAA - Another Weather Module

[cowboysdude]

@Nystro0m You have NO idea how much I appreciate that! THANK YOU and Tea is the nectar of the gods :)

[Nystro0m]

@cowboysdude No worries!
Im not sure about this one that @tbbear. So i leave that to you.

var curCon = document.createElement("div");
	curCon.classList.add("xsmall", "bright");
	curCon.innerHTML = this.translate("Currently: ") + this.translate(current.weather);
	wrapper.appendChild(curCon);

[cowboysdude]

@Nystro0m I will look at that this evening…Promise!

[Mykle1]

@Nystro0m said in MMM-NOAA - Another Weather Module:

I started of migrating all of my modules and main css-changes to a custom CSS.

You made main.css changes? Bad boy!

[Nystro0m]

@cowboysdude No worries! I also added the missing part in the Swedish translation.
Anything else i can help with?

[cowboysdude]

@Nystro0m Well if you have the winning lottery numbers that would be a big help LOL

No if you want feel free to look at it and if you have suggestions or changes let me know! :)

I started building this just for me…but after some requests I posted it… right now I"m working on turning the bottom forecast into a table so it looks better…

[Nystro0m]

@Mykle1 Well… You cant understand how everything works right away!
I though i needed to copy all CSS from main to custom. So i made a local backup of the Main.css instead.

I now know better! lol

[Mykle1]

@Nystro0m

He’s not even kidding. @cowboysdude is one of the hardest working people I know.

And then he does this for fun?

Sheeeesh!

[cowboysdude]

There in is MOST of my problem… I am HORRIBLE at CSS… just can’t seem to get it …

[Mykle1]

@cowboysdude said in MMM-NOAA - Another Weather Module:

Well if you have the winning lottery numbers that would be a big help LOL

I do have a Lottery module that might help you with that! ;^)

[cowboysdude]

@Mykle1 Yeah I remember that… LOL

[cowboysdude]

I apologize for this… I’m tired and memory is failing but… in the new version we only need a very minimal translation file… the weather will adjust automatically to the language you have selected in your config.js file ;) Here’s an example:

So there’s no need in adding to the translation files :) AND for this:

	curCon.classList.add("xsmall", "bright");
	curCon.innerHTML = this.translate("Currently: ") + current.weather;
	wrapper.appendChild(curCon);

Is correct because the variable current.weather will show in the chosen language all by itself :)

Hang in there I"m trying to finalize what I’m working on now.

[tbbear]

Thanks for the new translations, now only a few topics do not fit for german users.

1. we would not say “Anzahl der Tageslicht” we say “Stunden mit Tageslicht”
2. Not “11 Std. 48 Protokoll” we say “11 Std. 48 min”
3. we dont measure airpressure in “psi” like 29.99 here i also think this is same in a lot of countries our value is “hPa” which means “HectoPascal”
4. in your last post u said that my change is not necessary any more but when i remove it i still get only english actual weather so i keep it

U did and do a great Job and i hope u are not angry about my comments, but i only try to help.
So one last topic:
i saw an issue at github that people have problems installing the module and i think i know why.
If u look at your readme file it lookes like that the apiKey and the airKey are the same keys and thats wrong they are differen keys from different websites. U grab the useAir from “http://api.airvisual.com” so a user has to sign in there and get the airKey from this side but their is no hint in your readme about this.
Hope i could help.

[cowboysdude]

@tbbear NO I appreciate the feedback…I’m using Google translate and I’m sure they are wrong at times :)

I’ll get that corrected! :)

1. we would not say “Anzahl der Tageslicht” we say “Stunden mit Tageslicht” ~ FIXED
2. Not “11 Std. 48 Protokoll” we say “11 Std. 48 min” ~ FIXED
   we dont measure airpressure in “psi” like 29.99 here i also think this is same in a lot of countries our value is “hPa” which means “HectoPascal” ~ Best I can do is to use mb
   in your last post u said that my change is not necessary any more but when i remove it i still get only english actual weather so i keep it ~ Not sure what you mean here…

Updated Readme ;)

[tbbear]

@cowboysdude
So there’s no need in adding to the translation files :) AND for this:

curCon.classList.add("xsmall", "bright");
curCon.innerHTML = this.translate("Currently: ") + current.weather;
wrapper.appendChild(curCon);

Is correct because the variable current.weather will show in the chosen language all by itself :)

this i ment

[cowboysdude]

@tbbear Not sure this looks right to me…

I change this line in my config.js file and get what you’re seeing below
" language: ‘en’," to “language: ‘de’,”

[tbbear]

this looks good see what i get:

[tbbear]

oooo by the way i just see ur change in airpressure wow this is great

[cowboysdude]

@tbbear That I don’t understand… so what I would do … delete the MMM-NOAA directory and do another install like it was new… something is getting crossed up there… you’re missing files. The image I posted is the one that is new on github …

[tbbear]

ok, i will do and report soon thanks