-
Notifications
You must be signed in to change notification settings - Fork 33
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
Inkbird IBS-P01R Pool Thermometer #1149
Conversation
Inkbird IBS-P01R Pool Thermometer, Inkbird ITH-20R
add sensor to commandref
add testdata for protocol 123, Inkbird IBS-P01R Pool Thermometer
add definition protocol 123, Inkbird IBS-P01R Pool Thermometer, ITH-20R
add Inkbird IBS-P01R, ITH-20R
…HEM/RFFHEM into master_Inkbird-IBS-P01R_02
add Inkbird_IBS-P01R to commandref
…HEM/RFFHEM into master_Inkbird-IBS-P01R_02
Codecov Report
@@ Coverage Diff @@
## master #1149 +/- ##
==========================================
+ Coverage 66.92% 67.35% +0.42%
==========================================
Files 133 135 +2
Lines 9784 9838 +54
Branches 1572 1577 +5
==========================================
+ Hits 6548 6626 +78
+ Misses 1945 1916 -29
- Partials 1291 1296 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@sidey79 | @elektron-bbs kann es durchaus möglich sein, das hier bei den Tests etwas durch die Lappen ging? Hier gibt es einen Schlüssel welcher nur Bitte mal prüfen. |
Sieht so aus, ist schon seit Dezember so drin: 0cbecba |
Ja, da fehlt ein s |
14_SD_WS - removal of unnecessary details and fix testdata #1149 (comment)
What is the current behavior?
(You can also link to an open issue here, if this describes the current behavior)
No reception of this sensor, see https://forum.fhem.de/index.php/topic,128945.0.html.
What is the new behavior (if this is a feature change)?
Reception and evaluation of the sensor data works.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no
Other information: