DrozmotiX / ioBroker.coronavirus-statistics

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

New german vaccination source #275

Closed jlssmt closed 3 years ago

jlssmt commented 3 years ago

First implementation of new sources. Will continue, if you're happy with it ;-)

DutchmanNL commented 3 years ago

First implementation of new sources. Will continue, if you're happy with it ;-)

I am ! 🗡️ :)

DutchmanNL commented 3 years ago

PS @jlssmt test gehen schief weil die version in 3 Dateien angepasst werden muss

package.json package-lock.json io-package.json

jlssmt commented 3 years ago

@DutchmanNL

PS @jlssmt test gehen schief weil die version in 3 Dateien angepasst werden muss

package.json package-lock.json io-package.json

erledigt

DutchmanNL commented 3 years ago

@DutchmanNL

PS @jlssmt test gehen schief weil die version in 3 Dateien angepasst werden muss package.json package-lock.json io-package.json

erledigt

danke, PS uebersetzen und die versionen macht alles das release script ist auch in diesem adapter implementiert :)

jlssmt commented 3 years ago

@DutchmanNL ach witzig. danke also einfach nur englisch und deutsch angeben und die restlichen attribute/ sprachen erstellt das script?

DutchmanNL commented 3 years ago

also einfach nur englisch und deutsch angeben und die restlichen attribute/ sprachen erstellt das script?

jup EN reicht, das script ĂĽbersetzt dan alles was man in der readme hinterlegt hat automatisch in die news. Also im grĂĽnde reicht es die readme zu schreiben, version und news macht dan das script :)

jlssmt commented 3 years ago

@DutchmanNL

ahhh. die readme ist der trigger. joa. gut. die readme hab ich gar nicht angefasst :-D ich hole das beim nächsten PR nach

DutchmanNL commented 3 years ago

ich hole das beim nächsten PR nach

kannst das script local ausfuehren mit diesem commando (vorher npm i im adapter Verzeichnis ausfuehren)

npm run release prepatch -- --dry --all bitte immer als "pre" finals mache ich dan spetter beim mergen.

Dazu musst du dan in der readme das hier aufnehmen :

    ### __WORK IN PROGRESS__
    * (DeveloperName) 

kannst ja mal mit spielen beim lösen von meinen falschen merg 👯‍♂️

https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/blob/main/main.js#L351-L375

jlssmt commented 3 years ago

@DutchmanNL ich fix deinen merge fehler. das ist für mich ja einfacher, weils mein code ist. stell dann später nen neuen PR. kannst du mir dafür vllt mal beschreiben, wie du grob die module entwickelst bzw. testest? ich entwickel das grad bei mir auf dem PC und schieb dann die Main.js von hand auf den rpi (mit iobroker) und mach iobroker stop, upload start und teste es dann. das ist aber ziemlich nervig. vor allem brauch ich auch immer this.log.debug und muss die log von iobroker ziehen.

DutchmanNL commented 3 years ago

kannst du mir dafĂĽr vllt mal beschreiben, wie du grob die module entwickelst bzw. testest?

ich arbeite mit Webstrom und den eingebauten debugger + regte file system. Ergo ich habe einen Der-Ordner, da liegt Der Adapter drin.... die Dateien schiebe ich dan auf ein der system (Knopfdruck in Webstrom) und danach benutze ich den debugger und durchlaufe damit den code

iobroker stop, upload start

iobroker stop ist nicht nötig, nur Der Adapter selber.... upload nur wen du was in den HTML (also frontend) oder io-package geaendert hast

Apollon77 commented 3 years ago

Schaut Euch doch mal den dev-server an ... klappt geil (und spart so upload und sowas)

DutchmanNL commented 3 years ago

Schaut Euch doch mal den dev-server an ... klappt geil (und spart so upload und sowas)

jeder wier er mochte, ich habe einen Developer Umgebung mit daten die ich brauche das kan der den server nicht ;) (wiederhohlung..... fuer diesen adapter kan es praktisch sein ja hast recht)

jlssmt commented 3 years ago

@Apollon77 danke für den tipp. ich check ihn aber nicht. dev-server run funzt, aber dann läuft ja der adapter nicht. und dev-server debug bringt mir keine ausgabe oder evtl sogar nen cmd fehler. sind ziemlich viele outputs und keine ahnung, was da relevant ist.