`/api/tree` API applies unneeded time filter by default
Maintainer thường phản hồi trong vòng 2 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 65/100
Hướng nghiên cứu
Bắt đầu bằng cách lần theo endpoint /api/tree và các route / và /tree của dashboard, tập trung vào nơi interval_in_days nhận các giá trị mặc định. Xác nhận hành vi hiện tại và đảm bảo các giá trị mặc định không được thiết lập trên các entry point này, sau đó kiểm tra rằng các request không có tham số trả về tất cả các checkout mới nhất hiện có và redirect không có interval được hardcode.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Description
/api/tree queries for the latest checkout for each tree. By default, it filters by checkouts in the last 7 days. However, this operation is really fast now (queries a pre-computed table), and there should be no reason to have this limit by default.
Related to this, there is a misagreement between the API and the dashboard here:
- API default is
interval_in_days=7(as mentioned above) - The dashboard root
/redirects to/tree?i=7and queries withinterval_in_days=7(not present in production) - Dashboard
/treequeries withinterval_in_days=30
Proposed solution
Therefore, I suggest:
- Keep
interval_in_daysas optional, but do not filter if unset - Make
/redirect to/treewithout hardcoded parameters - Make
/treequery withinterval_in_daysunset by default- Alternatively, grab the default directly from the backend
- Ngôn ngữ chính
- Python
- Star
- 9
- Fork
- 31
- Merge trung bình
- 5 ngày 22 giờ
- Pull request đã merge (30 ngày)
- 19
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Không có mẫu pull request
- Đọc 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 kernelci/dashboard
-
4xx and 5xx by endpointCó thể đã có người làm @alanpeixinho đã nhận 4 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 2 ngày
-
[Local dev environment] PRIVACY.md missing in dashboard containerCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
kernelci/dashboard#2142 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 2 ngày
-
[AB Comparison] Add labels to Previous commit and Branch head buttonsCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
kernelci/dashboard#2128 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 52/100
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của kernelci/dashboard
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
-
Harmony OPeNDAP SubSetter (HOSS) Geographic LARC_CLOUD PREFIRE_SAT2_AUX-SAT R01 production
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
nasa/harmony-autotester#245 ·
-
[FEATURE] - Add UTVD supportĐang mởenhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Deltares/imod-python#1928 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100