-
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
New elements for attribute hardware #1172
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1172 +/- ##
==========================================
- Coverage 67.57% 66.99% -0.58%
==========================================
Files 135 134 -1
Lines 9812 9807 -5
Branches 1570 1570
==========================================
- Hits 6630 6570 -60
- Misses 1887 1942 +55
Partials 1295 1295
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 7 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
- updated Hardwarelist
8680875
to
03138c3
Compare
Mhmm, muss das mit dem angehangen "s" unbedingt sein? Müssen dann alle User das Attribut ändern? Außerdem gibt es noch etliche Differenzen zwischen Attribut, Dateiname und commandref. Das müsstest du dir nochmal genauer ansehen. Hier nur einige Beispiele:
Vielleicht auch noch Groß-/Kleinschreibung vereinheitlichen, z.B. cc1101 vs. CC1101? |
Ich sehe da aktuell keine einfache Alternative um eine Firmware eindeutig zu identifizieren. Betrifft hauptsächlich die ESPs welche sich ohnehin derzeit nicht über das Modul updaten lassen.
Da nano328 bereits eindeutig ist, können wir dies so belassen. Es erspart den Anwendern das Anpassen ihres Attributen.
Ich finde kein ESP32d.
Das würde zu weiteren Änderungen der Attribute führen. Bei der Suche nach der Firmware spielt groß/kleinschreibung keine Rolle. |
schreibweise aktualisiert
(You can also link to an open issue here, if this describes the current behavior)
Hardware for esp32 and esp8266 does not guarantee, that it will load the right firmware, because the hardware name is searched in the filename.
Hardware names without cc1101 have a suffix s which makes them uniqe
Hardware names are also adapted to be compatible with RFD-FHEM/SIGNALDuino#278
yes, the following hardware attribute are renamed
esp32 to esp32s
esp8266 to esp8266s
promini is removed and replaces with
promini8cc1101
promini16cc1101
promini8s
promini16s