[Script] DWD/UWZ/NINA Warnungen als Push/Sprachnachrichten

[sigi234]

@ticaki sagte in [Script] DWD/UWZ Warnungen als Pushnachrichten:

Was ich noch ändern möchte. Eure Meinung ist erwünscht.
TODO:

states auch unter 0_userdata anlegbar

Ja

das DWD und UWZ in nur 1 Scriptinstanz laufen. (gute Idee?)

Trennen

FRAGE:
Die Region ist bei UWZ nur als UWZDE12345 angegeben. Wenn ich da eine Regionsbezeichnung einbaue wird gibt es 3 Möglichkeiten.

Einen Teil von UWZDE12345 benutzen
Konfigurierbare Ausgabe z.B. ['UWZDE12345','Entenhausen'] wenn UWZ... gefunden wird gibt es Entenhausen aus
Es besteht bei euch kein Bedarf.

Konfigurierbare Ausgabe z.B. ['UWZDE12345','Entenhausen']

[sigi234]

@ticaki

Ansage von Alexa geht, aber sie sagt :
Achtung alle Warnmeldungen der ufs wurden aufgehoben.

[Latzi]

@sigi234 sagte in [Script] DWD/UWZ Warnungen als Pushnachrichten:

Achtung alle Warnmeldungen der ufs wurden aufgehoben.

Stimmt, klingt bei mir auch so.
Ich hab im Script das Ansagewort "der UWZ" durch " der Unwetterzentrale" ersetzt, klingt einfach besser
(Versuch mit "der UWeZet" klang auch etwas komisch)

[sigi234]

@Latzi sagte in [Script] DWD/UWZ Warnungen als Pushnachrichten:

ch hab im Script das Ansagewort "der UWZ" durch " der Unwetterzentrale" ersetzt, klingt einfach besser

Wo kann ich das ändern, dann muss ich nicht alles neu machen?

[Latzi]

@sigi234
so ca. Zeile 400 im script:

var artikelMODE = MODE == 'DWD'?'des DWD' : 'der UWZ';

durch

var artikelMODE = MODE == 'DWD'?'des DWD' : 'der Unwetterzentrale';

ersetzen.

[ticaki]

Zeile 410

v.0.89.2
UWZ durch Unwetterzentrale ersetzt
Aufgehoben "des DWD" korrigiert
UWZ-Headline verändert von "Warnung vor..." zu "Warnung der Unwetterzentrale vor..."

// Änderungen
410 var artikelMODE = MODE == 'DWD'?'des DWD' : 'der Unwetterzentrale';
446 let pushmsg = "Die Wetterwarnung " +"'"+ headline + " gültig bis " + end + "'" + " "+artikelMODE+" wurde aufgehoben.";
667 result['headline'] = warn.type === undefined ? '' : 'Warnung '+artikelMODE+' vor '+warningTypesString[result.type];
//

Und wenn ihr den Code ab Ende der Konfiguration kopiert braucht ihr nicht alles neu einzutragen. Ich mach das auch so.
Ich schreib in Atom und kopiere dann zum Testen rüber.

[ticaki]

@sigi234 sagte in [Script] DWD/UWZ Warnungen als Pushnachrichten:

@ticaki sagte in [Script] DWD/UWZ Warnungen als Pushnachrichten:

das DWD und UWZ in nur 1 Scriptinstanz laufen. (gute Idee?)

Trennen

Meinst damit

• 2 getrennte Scripte (sehr aufwendig zu pflegen)
• 2 Scripte in denen ich MODE fest vorgebe und im Code verstecke
• so wie es jetzt ist

?

[Latzi]

@ticaki persönlich würde ich es so belassen, ist doch nur ein Parameter der gesetzt werden muss.

[sigi234]

@Latzi sagte in [Script] DWD/UWZ Warnungen als Pushnachrichten:

@ticaki persönlich würde ich es so belassen, ist doch nur ein Parameter der gesetzt werden muss.

Ja hast Recht, belassen!

[ticaki]

An der aktuellen Version werde ich jetzt nichts mehr ändern außer Fehlerbehebungen, wenn da 1-2 Tage keine Fehlermeldung gepostet wird, kommt es aus dem Test Ordner und ich baue die offenen Dinge in die Testversion ein.

[CruziX]

@ticaki sagte in [Script] DWD/UWZ Warnungen als Pushnachrichten:

TODO:

• states auch unter 0_userdata anlegbar

Also da bin ich auf jeden Fall auch für (ja, äußerst überraschend )

[ticaki]

@CruziX

@all
aus einem mir noch nicht nachvollziehbaren Grund muß das hier eingefügt werden:
EDIT: Hab den Grund verstanden, bitte verschieben.

entfernen ca. 415ff
var artikelMode = MODE == 'DWD'?'des DWD' : 'der Unwetterzentrale';

nachher: ca. 150ff
var onClickCheckRun = false;
var warnDatabase = {new:[],old:[]};
var artikelMODE = MODE == 'DWD'?'des DWD' : 'der Unwetterzentrale';
    

sonst ist artikelMode undefiniert

[CruziX]

@ticaki Wenn ich das richtig sehe hast du auch 2 gleichnamige Variablen.
also einmal var artikelMode und einmal let artikelMode, könnte da das Problem liegen?

[ticaki]

@CruziX
Ne ich hatte nur var artikelMode. Aber die ist wenn man das Skript über "on" aufruft undefiniert... Hm vielleicht liegt es daran das ich artikelMode nicht am Anfang definiert hab.

Jep: Jep genau das war es.

[CruziX]

@ticaki Was war was? ^^
lag es an der Definition?

[ticaki]

@CruziX sagte in [Script] DWD/UWZ Warnungen als Pushnachrichten:

@ticaki Was war was? ^^
lag es an der Definition?

Ja, was auch logisch ist. Code oberhalb der Definition hat eine Funktion unterhalb aufgerufen.
EDIT: hab den Code oben aktualisiert

[ticaki]

ein weiterer Fehler:
Sprachausgabe: Wenn das Warnlevel höher als warnlevel war, wurde der mittlere Teil der Sprachausgabe weg gelassen.

Der Fehler war in allen Skripten

Hab die vom Funktionsumfang fertige Version im Test Ordner online gestellt.
Das neue Script hab ich mit Alexa, Telegram, eMail durchgetestet. Bis auf einzelne DWD Entwarnungen konnte ich alles testen.

Das neue hat

• 0_userdata
• Regionen Namen für UWZ
• Ein paar Konfigurationsprüfungen mehr.

[ticaki]

Hab ne Variante für Nina only im Testordner.

[crunchip]

@ticaki jetzt lässt es aber langsam Krachen
Nur mal eben nen Fehler beheben im script und dann sowas

[sigi234]

@ticaki

Warum habe ich Version 0.89.1 ?