iMicknl / iMicknl/python-sagemcom-api
Add support for TalkTalk-branded router (UK)
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 141
- Fork
- 49
- Merge trung bình
- 7 giờ 54 phút
- Pull request đã merge (30 ngày)
- 4
Mô tả
## 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]
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với đường dẫn GetDeviceInfo được mô tả trong issue và xem xét PR #386, vốn đã xử lý ModelNumber bị thiếu và bổ sung xử lý ActionError cho nhiều xpaths. Được xem là hoàn thành khi TalkTalk F5364 có thể hoàn tất thiết lập trong trường hợp không có ModelNumber mà không gặp exception đã nêu.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- networking
- Loại issue
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 20/100