JSON API mixes null and empty strings for missing URLs
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Bắt đầu bằng cách theo dõi quá trình tuần tự hóa API cho list_archive_url, webscm_url và các trường URL nullable khác được đề cập trong issue. Xác định xem hành vi giữa null và rỗng được DRF hay Patchwork kiểm soát, sau đó xác minh rằng các URL nullable luôn tạo ra null và hành vi này được bao phủ bởi các bài kiểm thử API liên quan.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
This is in quite a few places.
list_archive_url is null (though it can be "" too! not sure what the correlation is), most other nullable URLs i.e. webscm_url are "".
I believe the best option is to have them all be null, avoiding cases where a JSON parser knows a field should contain a URL, finds a value (since it's not null), and can't parse it as a valid URL.
Also not sure if this is a DRF quirk or something Patchwork can control.
- Ngôn ngữ chính
- Python
- Star
- 317
- Fork
- 91
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
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.
Issue khác của getpatchwork/patchwork
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
getpatchwork/patchwork#546 · 2 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 52/100
getpatchwork/patchwork#664 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
getpatchwork/patchwork#663 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
getpatchwork/patchwork#645 ·
Tất cả issue của getpatchwork/patchwork
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
learningequality/ricecooker#747 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
BSData/horus-heresy-3rd-edition#3171 ·
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
run-llama/llama_index#23199 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
KhronosGroup/glTF-Blender-IO#2769 ·