Mentorship: request lifecycle + capacity enforcement (Phase 6.3)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- csharp, postgresql
- Ambito
- backend-api-design, databases, testing
Direzione di ricerca
Inizia risolvendo dependency #9 e leggendo i pattern esistenti per gli endpoint dei moduli e i test harness, in particolare IEndpointModule.MapEndpoints e Members/MembersTestHarness.cs. Traccia come vengono usati Result, ProblemDetails, il contesto EF Core di proprietà del modulo e IEmailSender. Il lavoro è completato quando gli xUnit tests coprono l'intero ciclo di vita, le regole di autorizzazione, la race condition sulla capacità, la paginazione, gli errori stabili e un comportamento dell'email sicuro in caso di errore.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Scope
Mentee-initiated requests through to active mentorships.
Lifecycle
Pending → Accepted ⇒ ActiveMentorship | Declined(reason) | Withdrawn — then Active → Concluded (either side).
Data model
public sealed class MentorshipRequest
{
public Guid Id { get; set; }
public Guid MentorId { get; set; }
public Guid MenteeId { get; set; }
public string Message { get; set; }
public RequestStatus Status { get; set; } // Pending | Accepted | Declined | Withdrawn
public string? DeclineReason { get; set; }
}
public sealed class Mentorship
{
public Guid Id { get; set; }
public Guid MentorId { get; set; }
public Guid MenteeId { get; set; }
public MentorshipStatus Status { get; set; } // Active | Concluded
}
Endpoints (active-member policy)
| Method | Path | Notes |
|---|---|---|
| POST | /mentorship/requests |
mentee → mentor; message + goals |
| GET | /mentorship/requests/incoming / outgoing |
paginated |
| POST | /mentorship/requests/{id}/accept |
mentor only; creates Mentorship |
| POST | /mentorship/requests/{id}/decline |
mentor only; reason optional but stored |
| POST | /mentorship/requests/{id}/withdraw |
mentee only |
| GET | /mentorship/mine |
active + concluded mentorships for caller |
| POST | /mentorship/{id}/conclude |
either participant |
Rules
- Capacity on accept: active mentorships <
Capacity, else409 mentorship.at_capacity. Hitting capacity auto-setsIsAcceptingMentees = false(manual re-enable when a mentorship concludes — do not auto-flip back). - No duplicate open request to the same mentor; no self-mentorship; mentor must be accepting.
- Brevo (existing
IEmailSender, best-effort): request received → mentor; accepted/declined → mentee.
Error codes
mentorship.not_found, mentorship.at_capacity, mentorship.not_accepting, mentorship.duplicate_request, mentorship.invalid_state, mentorship.forbidden
Dependencies
Blocked by #9.
Acceptance criteria
- Full transition matrix tested incl. wrong-party actions (mentee accepting, etc.)
- Capacity race: two concurrent accepts at capacity-1 → exactly one succeeds (constraint or transaction, tested)
- Emails logged-not-thrown on failure
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).
- Lingua principale
- C#
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CommunityPro/community-pro-api
-
cross-cutting decision
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
cross-cutting
Difficoltà 4/5 3-5 giorni Idoneità per principianti 58/100
-
admin
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
admin
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
sponsorship
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
Tutte le issue di CommunityPro/community-pro-api
Issue simili
-
bug priority: P2
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Parameterless constructor is not a must-provided by user when a struct has a field initializerAperta:watch: Not Triaged dotnet-csharp/svc fundamentals/subsvc
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Wrong values of Kernel32.FileShare.Delete and Kernel32.FileAttributes.PosixSemanticsForse già presa @ElectLive l’ha presa oggi. Apertabug good first issue help wanted
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
ForNeVeR/TruePath#241 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
files-community/Files#19047 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno