New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Anzeige im WEBUI für Thermostat HM-CC-RT-DN erweitern #444

Closed
Bratmaxe1981 opened this Issue Oct 12, 2018 · 8 comments

Comments

4 participants
@Bratmaxe1981

Bratmaxe1981 commented Oct 12, 2018

Der User jp112sdl hat einen kleinen Patch für die Anzeige von Batteriezustand und Ventilposition des Heizkörperthermostat HM-CC-RT-DN veröffentlicht.

Es wäre schön, wenn dieser Patch Standard in Raspberrymatic werden könnte.
Hier der Link zu dem GitHub-Projekt: https://github.com/jp112sdl/HM-WebUI-CC-Mod

Es scheint nur eine kleine Anpassung in einer Datei notwendig zu sein.

@jens-maus

This comment has been minimized.

Owner

jens-maus commented Oct 12, 2018

Tut mir leid, aber wenn ich diesen Patch 1:1 übernehmen können sollte muss der Autor einer Veröffentlichung unter der Apache 2.0 Lizenz zustimmen da er momentan unter der GPL 3.0 Lizenz veröffentlich ist und diese beist sich mit Apache 2.0. Ansonsten müsste ich diesen komplett selbst neu entwickeln müssen und das kann dauern ;)

@jp112sdl Stimmst du einer Integration dieser Anpassung unter der Apache 2.0 Lizenz zu?

@jp112sdl

This comment has been minimized.

Contributor

jp112sdl commented Oct 12, 2018

Hi zusammen!

Ja, das ist kein Problem.
Die Modifikation kann gern übernommen werden.

Ich hatte bei der Wahl der GPL Lizenz den Schwerpunkt auf den Passus "Disclose source" gelegt.
Aber das ist ja hier gewahrt. Und letztlich bei der kleinen Modifikation auch nicht so wild.

Also 🔥 frei.

@jens-maus

This comment has been minimized.

Owner

jens-maus commented Oct 12, 2018

Alles klar, dann wird das für die nächste RaspberryMatic übernommen werden. Und in Zukunft bitte gleich ein PullRequest mit solchen WebUI Anpassungen stellen :) Daran bin ich immer interessiert...

@jens-maus jens-maus added this to the next release milestone Oct 12, 2018

@Bratmaxe1981

This comment has been minimized.

Bratmaxe1981 commented Oct 12, 2018

Da bedanke ich mich schon mal bei euch beiden.

@jens-maus jens-maus removed the undecided label Oct 12, 2018

@jens-maus jens-maus added this to In progress in WebUI improvements/fixes Oct 12, 2018

@jens-maus jens-maus closed this in 5c5ce50 Oct 12, 2018

@jens-maus jens-maus moved this from In progress to Done in WebUI improvements/fixes Oct 12, 2018

@jens-maus

This comment has been minimized.

Owner

jens-maus commented Oct 12, 2018

Voila, integriert!

@Bratmaxe1981

This comment has been minimized.

Bratmaxe1981 commented Oct 12, 2018

Perfekt danke, das nenne ich mal ne schnelle Anpassung!

@jp112sdl

This comment has been minimized.

Contributor

jp112sdl commented Oct 12, 2018

Top 👍

@MathiasJZ

This comment has been minimized.

MathiasJZ commented Oct 25, 2018

Cool! Ich freue mch schon drauf! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment