iMicknl / iMicknl/python-sagemcom-api
Add support for TalkTalk-branded router (UK)
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 141
- Forks
- 49
- Ø Merge
- 7 Std. 54 Min.
- Gemergte PRs (30 T.)
- 4
Beschreibung
Model information
F5364 from TalkTalk
Describe the bug
GetDeviceInfo is failing because this particular flavour of router doesn't have a ModelNumber.
Otherwise the library is working great!
Impact is that HA is unable to connect to the router since an exception is thrown in the setup.
Expected behavior
I think ModelNumber is not important in the bigger scheme and could be defaulted to blank.
Additional context
I've submitted a PR that fixes this - #386 - and also provides better support for ActionErrors when the request contains multiple xpaths.
[bug]
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem im Issue beschriebenen GetDeviceInfo-Pfad und prüfe PR #386, die das fehlende ModelNumber bereits behandelt und ActionError-Handling für mehrere xpaths ergänzt. Als abgeschlossen gilt die Aufgabe, wenn der TalkTalk F5364 die Einrichtung bei fehlendem ModelNumber ohne die gemeldete Exception abschließen kann.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- networking
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 20/100