Contributors: leaderboard endpoint with period filters (Phase 5.3)
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
- 45/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- csharp, postgresql
- Lĩnh vực
- backend-api-design, databases
Hướng nghiên cứu
Bắt đầu bằng cách kiểm tra dependency #6, sau đó lần theo pattern của endpoint module thông qua IEndpointModule.MapEndpoints và public contract hiện có của Members/Identity. Đọc các pattern của ContributionEvent và MembersTestHarness.cs trước khi thêm coverage của xUnit dưới tests/CommunityPro.Tests//. Hoàn tất nghĩa là có response ẩn danh được phân trang, thứ hạng xác định, coverage cho ranh giới kỳ theo UTC, ánh xạ lỗi ổn định và dữ liệu profile chỉ dành cho thành viên đang hoạt động.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Scope
GET /contributors?period=all|90d|30d (anonymous, paginated) returning the ranked leaderboard the Top Contributors page renders (ISR, revalidate 300s).
Shape
{ "rank": 1, "login": "octocat", "avatarUrl": "…", "memberId": "…|null", "displayName": "…|null", "mergedPrs": 42 }
Rules
- Aggregate with a plain grouped query over
ContributionEventfirst (materialize into acontributor_statsview/table only if it proves slow — don't pre-optimize). - Join to member profiles via
GithubUserIdthrough a public module contract on the Members/Identity side (e.g. a bulk lookupIReadOnlyDictionary<long, MemberRef>by GitHub id) — no cross-module DbContext access. - Non-member org contributors still appear (login + GitHub avatar,
memberId: null). Deactivated members appear as non-members (no profile link). - Period boundaries computed in UTC off
MergedAt.
Error codes
contributors.invalid_period
Dependencies
Blocked by #6. Backfill #7 makes the data real but isn't a code dependency.
Acceptance criteria
- Ranking is stable (ties broken deterministically, e.g. earliest merge first)
- Period filters tested at the boundary (e.g. a PR merged exactly 90d ago)
- Member join returns profile info only for active members
- Paginated; default page size sensible (~20)
Conventions (project-wide, non-negotiable)
- .NET 9, records for immutable shapes, file-scoped namespaces, primary constructors where they read well. Minimal-API endpoints grouped per module via
IEndpointModule.MapEndpoints. Result<T>(SharedKernel) instead of exception-driven control flow. Endpoint results map failures to ProblemDetails with the stable error codes listed above — the frontend keys off them.- Module owns its EF Core
DbContextmapped to its own Postgres schema. Modules never reference each other's internals — cross-module needs go through a public contract interface or an in-process domain event (IEventPublisher). - All external calls (GitHub, Stripe, Brevo, Cloudinary, Meilisearch) behind interfaces owned by the consuming module.
- Every list endpoint paginated (offset is fine). xUnit tests in
tests/CommunityPro.Tests/<Module>/following the existing harness patterns (seeMembers/MembersTestHarness.cs).
- Ngôn ngữ chính
- C#
- Star
- 0
- Fork
- 0
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Không có mẫu pull request
- Không 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 CommunityPro/community-pro-api
-
cross-cutting decision
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
cross-cutting
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 58/100
-
admin
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
admin
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
sponsorship
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
Tất cả issue của CommunityPro/community-pro-api
Issue tương tự
-
[Tool] DirectBenchĐang mởhas-image has-readme needs-attention new-tool repo-verified
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 62/100
shanselman/TinyToolTown#844 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 2 ngày
-
[i18n] 安装实例完成后的成功提示未正确本地化Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
PCL-Community/PCL-CE#3658 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Deploy & Patch-issues opprettes ikke: create-pnd-issues.yml har feilet hver uke siden 2025-09-08Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Altinn/altinn-auth#4359 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
アプリ: チャット 優先: 中 提案
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
yksr-melt/Meltype#243 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày