Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Wetterstation WH3080 / WH2315 - Forum#67587 #664

Closed
HomeAutoUser opened this issue Oct 13, 2019 · 2 comments
Closed

Wetterstation WH3080 / WH2315 - Forum#67587 #664

HomeAutoUser opened this issue Oct 13, 2019 · 2 comments
Assignees
@pejonp @sidey79 @Ralf9
Labels
question

Comments

@HomeAutoUser
Copy link
Contributor

HomeAutoUser commented Oct 13, 2019 •
edited

Specifications for new sensor / switch / or other device ...

@Ralf9 @pejonp
Da Ihr beiden hier https://forum.fhem.de/index.php/topic,67587.msg980092.html#msg980092 aktiv seit und eine modifizierte Version der 14_SD_WS09.pm in den Umlauf bringt, macht Ihr bitte auch hier auf Github den PR dazu um nicht automatisch zu einer ungewollten Differenz zu kommen. Es wäre schade wenn auf diesen Weg vielleicht etwas verloren geht da vielleicht parallel weiterentwickelt wird an einer anderen Stelle des Modules.

MfG

SD_WS09.txt
SD_WS09_Log.txt
@Ralf9
Copy link
Contributor

Ralf9 commented Oct 14, 2019

Ich habe eigentlich nicht vor einen PR auf Github zu machen.
Dies ist nur ein Vorschlag für pejonp. Ob er es so übernimmt oder es modifiziert, möchte ich Ihm überlassen, es ist sein Modul.

@HomeAutoUser
Copy link
Contributor Author

Es wäre schön, wenn wir hier eine Lösung finden, da erneut die Benutzer anfragen.
https://forum.fhem.de/index.php/topic,67587.msg1015862.html#msg1015862

HomeAutoUser added a commit to HomeAutoUser/RFFHEM that referenced this issue Jan 24, 2020
@HomeAutoUser
Update 14_SD_WS09.pm
42965e5 
- RFD-FHEM#664
- takeover
-> https://forum.fhem.de/index.php/topic,67587.msg980092.html#msg980092
-> https://forum.fhem.de/index.php/topic,67587.msg1016075.html#msg1016075
-> https://forum.fhem.de/index.php/topic,67587.msg1015862.html#msg1015862
@HomeAutoUser HomeAutoUser mentioned this issue Jan 24, 2020
Merged
3 tasks
HomeAutoUser added a commit that referenced this issue Jan 29, 2020
@HomeAutoUser @sidey79
Update 14_SD_WS09.pm #664 - Forum (#785)
0cdf6a0 
* Update 14_SD_WS09.pm
- #664
- takeover
-> https://forum.fhem.de/index.php/topic,67587.msg980092.html#msg980092
-> https://forum.fhem.de/index.php/topic,67587.msg1016075.html#msg1016075
-> https://forum.fhem.de/index.php/topic,67587.msg1015862.html#msg1015862

* Update test_SD_WS09_devices-definition.txt
- revised test to view more information

* cpanfile
add digest:crc

* Update test_SD_WS09_devices-definition.txt

* Update 14_SD_WS09.pm
- revised module
- fix PERL WARNING
- Corrected line indents
- fix, WindDirAverage (Reading windDirectionAverage)
- revised commandref

Co-authored-by: sidey79 <7968127+sidey79@users.noreply.github.com>
@HomeAutoUser HomeAutoUser changed the title Wetterstation WH3080 - Forum#67587 Wetterstation WH3080 / WH2315 - Forum#67587 Feb 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pejonp pejonp

@sidey79 sidey79

@Ralf9 Ralf9

Labels
question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pejonp @sidey79 @Ralf9 @HomeAutoUser