iMicknl / iMicknl/python-sagemcom-api
Add support for TalkTalk-branded router (UK)
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 141
- Forks
- 49
- Merge medio
- 7 h 54 min
- PR fusionados (30 d)
- 4
Descripción
## 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]
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con la ruta GetDeviceInfo descrita en el issue y revisa PR #386, que ya aborda la ausencia de ModelNumber y añade manejo de ActionError para varios xpaths. Se considera terminado cuando el TalkTalk F5364 puede completar la configuración aunque falte ModelNumber, sin la excepción indicada.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- networking
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 20/100