@gonyeahialam In estimation today, we came up with the following acceptance criteria: "Initial load is 20 participants, each subsequent scroll is also 20." Let us know if that is okay or if you have any thoughts. Thanks!

There is an issue to display the Date and time as is it on the wireframes, I am sending below the link with the format options we have available at the moment:
https://www.mediawiki.org/wiki/Manual:WfTimestamp#:~:text=wfTimestamp()%20(part%20of%20GlobalFunctions,RFC%202822%20format%20and%20more.

@gonyeahialam @ifried @Daimona

In T308576#8029405, @cmelo wrote:

There is an issue to display the Date and time as is it on the wireframes, I am sending below the link with the format options we have available at the moment:
https://www.mediawiki.org/wiki/Manual:WfTimestamp#:~:text=wfTimestamp()%20(part%20of%20GlobalFunctions,RFC%202822%20format%20and%20more.
@gonyeahialam @ifried @Daimona

Also noting that this is just for V0, and until T21992 is resolved (which is perhaps not easy).

Change 809008 had a related patch set uploaded (by Cmelo; author: Cmelo):

[mediawiki/extensions/CampaignEvents@master] Add the ability to load more participants on scroll

https://gerrit.wikimedia.org/r/809008

Test wiki created on Patch demo by CMelo (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/db24d0734b/w/

Test wiki created on Patch demo by CMelo (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/ede8054ee2/w/

Change 813640 had a related patch set uploaded (by Cmelo; author: Cmelo):

[mediawiki/extensions/CampaignEvents@master] Add the ability to load more participants on scroll

https://gerrit.wikimedia.org/r/813640

Change 813640 abandoned by Daimona Eaytoy:

[mediawiki/extensions/CampaignEvents@master] Add the ability to load more participants on scroll

Reason:

https://gerrit.wikimedia.org/r/813640

Test wiki on Patch demo by CMelo (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/db24d0734b/w/

(Moving back to in progress for the API docs update)

Change 809008 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] Add the ability to load more participants on scroll

https://gerrit.wikimedia.org/r/809008

Infinite scroll is working correctly, bringing back 20 users at a time at Special:EventDetails/{id} for all users (organizers, participants, and people who happen to come upon the page)

A couple of things that I think could be improved that are not in the AC of this ticket but we might want to make tickets for if we don't want to fix them in the scope of this ticket @ifried. The first item I feel would be a helpful improvement, and for the second I am not sure how much effort it would be from dev, so it may or may not be worth it :

1. It would be good to have some sort of a spinner or progress bar at the bottom of the scroll that displays while more participants are actively being loaded in, to show the user that more participants are being loaded into the list. @gonyeahialam do you have thoughts on this?
2. There is an unnecessary API call at the end of the scroll. For example, in the video clip above, when it reaches the end of the entire list of 42 participants, it does one more API call that brings back an empty array, even though there were only 2 participants loaded in in the prior call. That call is not needed because all of the participants have already been loaded into the UI. See below screenshot:

Also, for the following AC: Documentation of the "list participants" API endpoint is updated with the new request parameters and response data

@cmelo The docs for this endpoint at https://www.mediawiki.org/wiki/Extension:CampaignEvents/Api#List_the_participants_of_a_registration need to be updated to reflect the last_participant_id query param that is being used. Please tag me on this ticket when you get a chance to update that.

In T308576#8110591, @vaughnwalters wrote:

Infinite scroll is working correctly, bringing back 20 users at a time at Special:EventDetails/{id} for all users (organizers, participants, and people who happen to come upon the page)

A couple of things that I think could be improved that are not in the AC of this ticket but we might want to make tickets for if we don't want to fix them in the scope of this ticket @ifried. The first item I feel would be a helpful improvement, and for the second I am not sure how much effort it would be from dev, so it may or may not be worth it :

1. It would be good to have some sort of a spinner or progress bar at the bottom of the scroll that displays while more participants are actively being loaded in, to show the user that more participants are being loaded into the list. @gonyeahialam do you have thoughts on this?
2. There is an unnecessary API call at the end of the scroll. For example, in the video clip above, when it reaches the end of the entire list of 42 participants, it does one more API call that brings back an empty array, even though there were only 2 participants loaded in in the prior call. That call is not needed because all of the participants have already been loaded into the UI. See below screenshot:

---

Also, for the following AC: Documentation of the "list participants" API endpoint is updated with the new request parameters and response data

@cmelo The docs for this endpoint at https://www.mediawiki.org/wiki/Extension:CampaignEvents/Api#List_the_participants_of_a_registration need to be updated to reflect the last_participant_id query param that is being used. Please tag me on this ticket when you get a chance to update that.

Thank you so much @vaughnwalters, I will update the API doc!

Hi @vaughnwalters I already updated the API doc, thanks.

About the Item 1:
I think it would be a good improvement of usability too, what do you think @ifried and @gonyeahialam?

About item 2:
I will fix it thanks

Change 825381 had a related patch set uploaded (by Cmelo; author: Cmelo):

[mediawiki/extensions/CampaignEvents@master] remove unnecessary API call when loading more participants

https://gerrit.wikimedia.org/r/825381

In T308576#8175208, @cmelo wrote:

Hi @vaughnwalters I already updated the API doc, thanks.

About the Item 1:
I think it would be a good improvement of usability too, what do you think @ifried and @gonyeahialam?

About item 2:
I will fix it thanks

Hi @vaughnwalters, I removed the unnecessary API call, this is the gerrit link of the patch

Unnecessary API call is removed, and API docs updated to fix QA notes: T308576#8110591 👍

@vaughnwalters Apparently this task ended up in QA before CR. We made a small change to the patch, could you please test it again to make sure that it's still working? Thanks!

Change 825381 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] remove unnecessary API call when loading more participants

https://gerrit.wikimedia.org/r/825381

Infinite scroll is functioning correctly as seen in gif and network tab.

One bug though @cmelo see that the Usernames of participants will appear here when they register text and the graphic are still showing even after multiple users have registered. This exists on local as well

https://en.wikipedia.beta.wmflabs.org/wiki/Special:EventDetails/118

Also, on https://en.wikipedia.beta.wmflabs.org/wiki/Event:81 it shows 0 participants. This is out of scope for this ticket, but could be related to the bug mentioned above?

In T308576#8183358, @vaughnwalters wrote:

One bug though @cmelo see that the Usernames of participants will appear here when they register text and the graphic are still showing even after multiple users have registered. This exists on local as well

Yup, I've seen that and pushed a fix already: T316153.

Also, on https://en.wikipedia.beta.wmflabs.org/wiki/Event:81 it shows 0 participants. This is out of scope for this ticket, but could be related to the bug mentioned above?

No, the one above is a simple CSS bug. I see "47 participants" on that page, though.

In T308576#8183622, @Daimona wrote:

In T308576#8183358, @vaughnwalters wrote:

One bug though @cmelo see that the Usernames of participants will appear here when they register text and the graphic are still showing even after multiple users have registered. This exists on local as well

Yup, I've seen that and pushed a fix already: T316153.

Great 👍

Also, on https://en.wikipedia.beta.wmflabs.org/wiki/Event:81 it shows 0 participants. This is out of scope for this ticket, but could be related to the bug mentioned above?

No, the one above is a simple CSS bug. I see "47 participants" on that page, though.

Yep, I am as well now - maybe a cache or something.

Seems to be working well. I will be moving this to product sign-off.

It would be good to have some sort of a spinner or progress bar at the bottom of the scroll that displays while more participants are actively being loaded in, to show the user that more participants are being loaded into the list. @gonyeahialam do you have thoughts on this?

@vaughnwalters wrt the above, I can create a separate task for it. I would like a visual of how it currently looks when it's loading more users, I didn't notice any loading time/delay from my end, it seemed instantaneous.

In T308576#8186305, @gonyeahialam wrote:

Seems to be working well. I will be moving this to product sign-off.

It would be good to have some sort of a spinner or progress bar at the bottom of the scroll that displays while more participants are actively being loaded in, to show the user that more participants are being loaded into the list. @gonyeahialam do you have thoughts on this?

@vaughnwalters wrt the above, I can create a separate task for it. I would like a visual of how it currently looks when it's loading more users, I didn't notice any loading time/delay from my end, it seemed instantaneous.

@gonyeahialam check this page on desktop and especially on phone - I thought there was enough lag that a quick spinner for loads would be helpful, particularly on places that might have slower internet speeds, but let me know what you think: https://en.wikipedia.beta.wmflabs.org/wiki/Special:EventDetails/118

here's an example where I throttle the load speed down to low tier phone:

@gonyeahialam here's a good visual I just came across where a loading state would be useful - same event but on my phone when i load it through arabic wikipedia on betacluster.
https://ar.wikipedia.beta.wmflabs.org/wiki/Special:EventDetails/118

Test wiki on Patch demo by CMelo (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/ede8054ee2/w/