iMicknl / iMicknl/python-sagemcom-api
Add support for TalkTalk-branded router (UK)
还没有人认领这个 Issue。
- 主要语言
- Python
- 星标
- 141
- 派生
- 49
- 平均合并
- 7 小时 54 分钟
- 30 天内合并 PR
- 4
描述
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]
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
调研方向
从 issue 中描述的 GetDeviceInfo 路径开始，并检查 PR #386；该 PR 已处理缺失的 ModelNumber，并为多个 xpath 添加了 ActionError 处理。当 TalkTalk F5364 在缺少 ModelNumber 的情况下能够完成设置，且不会出现报告的 exception 时，即表示完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- python
- 领域
- networking
- Issue 类型
- 缺陷
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 活跃度
- 停滞
- 描述清晰度
- 描述清楚
- 新手友好度
- 20/100