PATCH /profiles/current lets a participant set their participantType
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 68/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- authorization
Direzione di ricerca
Individua il tipo di richiesta usato da PATCH /profiles/current e la definizione di UpdateProfileRequest. Inizia seguendo la route fino alla convalida della richiesta e confrontala con la route di amministrazione PATCH /profiles/{profileId}. Assegna alla route dei partecipanti un tipo di richiesta che escluda participantType; il lavoro è completato quando gli aggiornamenti dei partecipanti non possono modificare quel campo, mentre quelli di amministrazione possono ancora farlo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
PATCH /profiles/current writes the request body straight into the participant's profile, and UpdateProfileRequest includes participantType. So a participant can make themselves STANDARD, GUARDIAN, DEPENDENT_AGE or DEPENDENT_OTHER. The profile edit page doesn't send this field, so it takes a hand-made request.
participantType decides whose survey answers set a dependent's consent: a guardian's answers are copied to the family's dependents, combined across co-guardians. The admin route, PATCH /profiles/{profileId}, guards this. It refuses to leave a dependent with no guardian or to add a dependent to a family with no guardian, and it recalculates the dependents' answers afterwards. The participant route does none of that. A guardian can step out of their family's consent, or a participant can mark themselves a dependent in a family with no guardian, and the dependents' answers aren't recalculated either way.
Fix: give PATCH /profiles/current its own request type without participantType, and leave type changes to the admin route.
Related: #580, #585.
- Lingua principale
- TypeScript
- Stelle
- 4
- Fork
- 0
- Merge medio
- 14h 50m
- PR unite (30g)
- 5
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la 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 Garvan-Data-Science-Platform/ctrl
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 82/100
-
admin-client backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
backend bug enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Garvan-Data-Science-Platform/ctrl#956 · 1 commento ·
Tutte le issue di Garvan-Data-Science-Platform/ctrl
Issue simili
-
useFragment does not require `variables` when the fragment's variables type has required fieldsAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
apollographql/apollo-client#13493 ·
I maintainer di solito rispondono entro 1 giorno
-
Custom hosts written as trailing-dot FQDNs (`host.local.`) aren't detected as local backendsForse già presa @roboomp l’ha presa oggi. Apertabug prio:p2 providers triaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
can1357/oh-my-pi#15798 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
code-yeongyu/oh-my-openagent#10172 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
PDFCraftTool/pdfcraft#189 ·
-
Add the Yahoo chart API duplicate last-day row (yfinance#573) to incidents.md (about 1 hour)Forse già presa Una pull request collegata a questa issue è aperta o già unita. Apertadata docs good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 80/100
pacocartones/market-data-sanity-checker#160 ·
I maintainer di solito rispondono entro 1 giorno