dehsgr / homebridge-mobilealerts

This is a homebridge plugin for several Mobile-Alerts (Technoline) devices
ISC License
9 stars 1 forks source link

Added Support for MA10230 and pollingInterval Parameter and clean Umlaute #12

Closed Mart-Muc closed 4 years ago

Mart-Muc commented 4 years ago

Hi Sebastian, ich hoffe ich habe das richtige Vorgehen gewählt.

kannst du die 3 Sachen bitte übernehmen, Danke: Alles schon auf meinem System getestet. Zum selber Testen: Sensor-ID: 126654562F2E

Danke, Mart

dehsgr commented 4 years ago

Hallo @Mart-Muc ,

um Deinen Pull Request einfließen zu lassen möchte ich Dich bitten, im Vorfeld noch die folgenden Punkte zu berücksichtigen:

Das meiste Ist wie Du siehst nur Kosmetik - so bleibt der Code schön lesbar für alle. ;-) Tolle Arbeit von Dir! Da bin ich schon auf Deine Fakegato-Implementierung gespannt. ;-)

Mart-Muc commented 4 years ago

wie fetch ich denn? kann ich nirgends sehen. auf der github homepage? im github desktop programm? die konflikte kann ich nicht auflösen, da hierfür schreibrechte nötig sind ?!?

dehsgr commented 4 years ago

Das sollte Dir helfen: https://help.github.com/de/github/collaborating-with-issues-and-pull-requests/syncing-a-fork

Wenn nicht, löse ich den Konflikt mit Deinem nächsten Commit. ;-)

Mart-Muc commented 4 years ago

geht's jetzt zu mergen? ich hoff ich hab jetzt das fetchen verstanden...

einrücken hab ich mein bestes gegeben- ich arbeite mit notepad, also primitivst händisch, deswegen dürften deine blocks nicht verrutscht sein.

das mit dem Großbuchstaben im Parameternamen hab ich aus anderen Plugin übernommen.

schönen Abend:-)