Contributors: leaderboard endpoint with period filters (Phase 5.3)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- csharp, postgresql
- Área
- backend-api-design, databases
Línea de trabajo
Empieza comprobando dependency #6 y, después, sigue el patrón de módulos de endpoint a través de IEndpointModule.MapEndpoints y del contrato público existente de Members/Identity. Lee los patrones de ContributionEvent y MembersTestHarness.cs antes de añadir cobertura de xUnit en tests/CommunityPro.Tests//. Se considera terminado cuando haya una respuesta anónima paginada, un ranking determinista, cobertura de los límites de periodo en UTC, un mapeo estable de errores y datos de perfil únicamente de miembros activos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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).
- Lenguaje dominante
- C#
- Estrellas
- 0
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de CommunityPro/community-pro-api
-
cross-cutting decision
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
cross-cutting
Dificultad 4/5 3-5 días Aptitud para principiantes 58/100
-
admin
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
admin
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
sponsorship
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Todos los issues de CommunityPro/community-pro-api
Issues similares
-
type/automation type/tech-debt
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
High-DPI fixes for release/1.3: editor toolbar icons and Color Picker layout (patch included)Abiertono-stack-trace
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
v9 review: TestingAbiertodocs/external squad/utforming
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Altinn/altinn-studio#21041 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
stryker-mutator/stryker-net#3892 ·
Los mantenedores suelen responder en 1 día