| Daimona | |
| Oct 25 2022, 8:43 PM |
| F35704435: Screen Shot 2022-11-03 at 5.18.02 PM.png | |
| Nov 3 2022, 10:21 PM |
| F35704405: Screen Shot 2022-11-03 at 5.08.30 PM.png | |
| Nov 3 2022, 10:21 PM |
| F35704320: Screen Shot 2022-11-03 at 4.41.26 PM.png | |
| Nov 3 2022, 10:21 PM |
| F35704283: Screen Shot 2022-11-03 at 4.31.06 PM.png | |
| Nov 3 2022, 10:21 PM |
| F35704275: Screen Shot 2022-11-03 at 4.29.31 PM.png | |
| Nov 3 2022, 10:21 PM |
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add support for private registration to ListParticipantsHandler | mediawiki/extensions/CampaignEvents | master | +97 -21 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | MHorsey-WMF | T316405 [EPIC] Allow participant option of joining event privately | |||
| Resolved | Daimona | T319453 Implement support for private registration UI in EventDetails | |||
| Resolved | Daimona | T321625 Add support for privately registered participants to the "List the participants of an event" API endpoint |
Change 849186 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):
[mediawiki/extensions/CampaignEvents@master] Add support for private registration to ListParticipantsHandler
Change 849186 merged by jenkins-bot:
[mediawiki/extensions/CampaignEvents@master] Add support for private registration to ListParticipantsHandler
If query param ?include_private=false, then only publicly registered participants display:
If query param ?include_private=true is included, but user without permission to see privately registered users is logged in, then a 403 error is thrown. Note that this 403 is thrown even if the only privately registered user for the event is the one who is logged in.
If query param ?include_private=true is included, and organizer is logged in, then participants are displayed (including privately registered participants):
Also even if the user logged in is an admin but is not the organizer of the event, they get the 403 error:
@Daimona As a housekeeping note, I suggest moving the include_private param underneath id as to keep the required params together at the top in the endpoint doc for this:
Everything is working correctly so I am marking this as done/resolved ✅