Source version concept and mapping lists: num_found and next ignore updatedSince

Đang mở
  • Có gì không ổn?
#2,939 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

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
72/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
api

Hướng nghiên cứu

Start with the concept and mapping list views named in the issue: core/concepts/views.py around line 232 and core/mappings/views.py around line 100. Trace how total_count is calculated versus the filters used for the returned rows, then find the relevant tests for version list pagination. Done when tests show num_found, pages, and next reflect the filtered rows for updatedSince, with no next after the rows are exhausted.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

component/api decision/ai-ready signal/low-risk signal/quick-win signal/small-scope signal/well-specified stage/triaged type/bug

Summary

On a source version's concepts or mappings list, updatedSince (and updatedBy) filter the rows, but num_found, pages and next count the whole version. Example:

GET /orgs/CIEL/sources/CIEL/v2026-09-24/concepts/?updatedSince=2026-10-01&limit=100 → body [], num_found: 55582, num_returned: 0, and a next header through page 556.

A client that follows next, as the paging docs say to, makes 556 requests that each return nothing. Reproduced on staging: a 3-concept version with updatedSince after its release returned num_found: 3 and an empty body.

Cause: for non-HEAD versions, total_count counts through_qs with only the retired filter, while the rows use every filter from get_filters_for_criterion: core/concepts/views.py#L232, core/mappings/views.py#L100.

Acceptance criteria

  • On a version's concepts and mappings lists, num_found, pages and next count the same filtered rows the body pages through.
  • A test covers updatedSince on a version list: the headers match the rows, and there's no next when the rows are exhausted.
Ngôn ngữ chính
Không có dữ liệu ngôn ngữ
Star
4
Fork
2
Merge trung bình
2 giờ 12 phút
Pull request đã merge (30 ngày)
1

Chuẩn bị môi trường

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.

Issue khác của OpenConceptLab/ocl_issues

Tất cả issue của OpenConceptLab/ocl_issues

Issue tương tự

Thêm issue về Backend & API Design

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.