search

DrozmotiX / ioBroker.coronavirus-statistics

Adapter to show Global Corona Virus information and current reports
MIT License
18 stars 19 forks source link

New vaccination source for all countries #280

Closed jlssmt closed 3 years ago

jlssmt commented 3 years ago

@DutchmanNL ok. ich bin jetzt etwas eskaliert :-D Impfungen für alle Länder geht. Hab es mal als Funktion ausgelagert, die ein Land als String entgegen nimmt und dann die Daten zurückgibt. Die Daten schieb ich dann weiter in ne andere Funktion, die dann die Unterordner anlegt.

Ich hab nur am Ende gemerkt, dass ich jetzt vllt. noch was im frontend machen müsste. Darauf hab ich aber gar keinen Bock. Also bekommt jetzt jedes Land, das aktiviert ist, auch die Impfdaten. Egal ob man will oder nicht ;-)

Bitte den anderen PR mit dem hotfix von mir vorher mergen. Ich denk ich muss dann den master hier nochmal mergen.

DutchmanNL commented 3 years ago

@jlssmt :). danke ! ich bin der Meinung das wen man das. land aktiviert. auch die. Impfdaten kommen sollte. also passt so Habe den andere PR gemerged, bitte einen Rebase machen damit es wieder übereinstimmt dan koennen wir diesen mergen :)

DutchmanNL commented 3 years ago

@jlssmt merge conflicts solved, lets try :)

jlssmt commented 3 years ago

@DutchmanNL hmmm. jetzt haben wir doppelte arbeit gemacht. war grad drüber :-D hast du meinen kommentar oben gelesen, dass ich den Ordner _Impfungen zu Vaccination umbenannt hab?

jlssmt commented 3 years ago

Das sollte glaube ich noch unten bei cleanupolddata rein:

        await this.localDeleteState(`Germany._Impfungen.rkiImpfquote`);
        await this.localDeleteState(`Germany._Impfungen.rkiImpfungenKumulativTotal`);
        await this.localDeleteState(`Germany._Impfungen.rkiZweitImpfungenDifferenzVortag`);
        await this.localDeleteState(`Germany._Impfungen.rkiZweitImpfungenKumulativ`);
        await this.localDeleteState(`Germany._Impfungen.rkiImpfungenGesamtModerna`);
        await this.localDeleteState(`Germany._Impfungen.rkiImpfungenGesamtAstraZeneca`);
        await this.localDeleteState(`Germany._Impfungen.rkiImpfungenGesamtBioNTech`);
DutchmanNL commented 3 years ago

@DutchmanNL hmmm. jetzt haben wir doppelte arbeit gemacht. war grad drüber :-D hast du meinen kommentar oben gelesen, dass ich den Ordner _Impfungen zu Vaccination umbenannt hab?

jup gesehen, sorry hatte nicht beachten das du so schnell reagierst und dachte mach mal :)

das umbenennen gesehen, auch richtig so da jetzt international. Stimmt die edelstes fehlen noch nehme ich im niesten comment mit muss die bundeslaender noch fixen

jlssmt commented 3 years ago

@DutchmanNL kannst ja nicht wissen, dass ich 4 wochen vorm pc saß und F5 gedrückt hab, bis mein Hotfix durch war ;-) passt. danke für den merge. dann kann ich auch endlich wieder dein offizielles repo bei mir productive schalten :-D

mal was anderes in eigener sache. kennst du dich damit aus? https://forum.iobroker.net/topic/46767/error-adapter-startup-interrupted-with-code-11