iMicknl / iMicknl/python-sagemcom-api
Add support for TalkTalk-branded router (UK)
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 141
- Forks
- 49
- Merge moyen
- 7 h 54 min
- PR mergées (30 j)
- 4
Description
## 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]
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le chemin GetDeviceInfo décrit dans l’issue et examinez PR #386, qui traite déjà l’absence de ModelNumber et ajoute la gestion de ActionError pour plusieurs xpaths. C’est terminé lorsque le TalkTalk F5364 peut terminer la configuration en l’absence de ModelNumber, sans l’exception signalée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- networking
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 20/100