iMicknl / iMicknl/python-sagemcom-api

Add support for TalkTalk-branded router (UK)

Đang mở
#387 2 bình luận 1 reaction 0 người được giao Xem trên GitHub

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

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. 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

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.