Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#16 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

@bercedev đang làm issue này rồi.

Từ ngày 21/10/2026.

  • #23 của @bercedev — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
88/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
csharp
Lĩnh vực
api

Hướng nghiên cứu

Đọc QuickbaseNet/Helpers/QuickbaseQueryBuilder.cs, QuickbaseNet/Models/Options.cs và QuickbaseNet/Requests/QuickbaseQueryRequest.cs, sau đó xem các bài kiểm thử hiện có của trình tạo truy vấn. Triển khai các phương thức trình tạo được yêu cầu và hành vi tuần tự hóa tùy chọn, giữ Options là null nếu không gọi phương thức nào trong hai phương thức. Chạy dotnet test --filter FullyQualifiedName~QuickbaseQueryBuilderTests và xác nhận ba điều kiện hoàn tất được liệt kê.

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

Mô tả

enhancement good first issue

The query endpoint takes skip and top options for paging. QuickbaseQueryRequest already has an Options property for them, but QuickbaseQueryBuilder never sets it. This is the paging item in ROADMAP.md.

What to do

  • Add Skip(int count) and Top(int count) to QuickbaseQueryBuilder. Each returns the builder, like the other methods.
  • In Build(), set Options only when one of them was called. Otherwise leave it null, as it is today.
  • In Options, add DefaultValueHandling = DefaultValueHandling.Ignore to the skip and top properties, so calling only Skip(100) doesn't also send "top": 0.
  • Add XML doc comments to both methods.

QueryAllRecords is a separate roadmap item. Leave it out.

Where to look

  • QuickbaseNet/Helpers/QuickbaseQueryBuilder.cs (Build() is at line 86)
  • QuickbaseNet/Models/Options.cs
  • QuickbaseNet/Requests/QuickbaseQueryRequest.cs, line 46

Done when

  • .Skip(100).Top(50) builds a request with Options.Skip 100 and Options.Top 50
  • A request built without either has Options == null
  • JsonConvert.SerializeObject of a request built with only .Skip(100) contains "skip":100 and no "top" key
  • Tests for all three in QuickbaseNet.UnitTests/Tests/QuickbaseQueryBuilderTests.cs (create it if it doesn't exist)

Testing it

dotnet test --filter FullyQualifiedName~QuickbaseQueryBuilderTests
Ngôn ngữ chính
C#
Star
3
Fork
2
Merge trung bình
3 giờ 14 phút
Pull request đã merge (30 ngày)
2

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 hazeliscoding/quickbase-net

Tất cả issue của hazeliscoding/quickbase-net

Issue tương tự

Thêm issue về C#

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.