Page MenuHomePhabricator

Add chat group links in event details pop-up
Closed, ResolvedPublic3 Estimated Story Points

Description

As an event participant, I want to see the link to the chat group in 'More Details,' so I can easily access the information in the modal without needing to go to the EventDetails page.

Background: The chat group link is currently only viewable to participants in EventDetails (but not in 'More Details'). We should make it viewable in 'More Details' as well.

Acceptance Criteria:

  • Display link to chat group to users (if they have registered as participants or if they are organizers) below 'Location' information in 'more details' pop-up
  • The chat group information should be displayed in the same format/structure as it is displayed in EventDetails
  • Chat group links should only be displayed to participants after someone registers for event

Visual examples of current behavior:

I don't see chat link as a participant on More Details:

Screen Shot 2022-08-24 at 8.06.20 AM.png (800×1 px, 107 KB)

I see chat link as a participant in EventDetails:

Screen Shot 2022-08-24 at 8.06.36 AM.png (852×1 px, 149 KB)

Visual examples of proposed behavior:

Screenshot 2022-08-25 at 17.43.10.png (1×3 px, 663 KB)

Event Timeline

ifried updated the task description. (Show Details)
ifried updated the task description. (Show Details)

@gonyeahialam Hello! Is there an updated design/image we could use for this ticket? Thank you!

ifried renamed this task from Add chat group links in event details pop-up [VO or V1] to Add chat group links in event details pop-up.Aug 24 2022, 12:21 PM
ldelench_wmf set the point value for this task to 3.

Change 826896 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[mediawiki/extensions/CampaignEvents@master] Show group chat invite link in the "more details" dialog

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

Change 826896 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] Show group chat invite link in the "more details" dialog

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

  • Display link to chat group to users (if they have registered as participants or if they are organizers) below 'Location' information in 'more details' pop-up

Participant:

Screen Shot 2022-08-31 at 11.25.32 AM.png (1×1 px, 652 KB)

Organizer (same view as participant):
Screen Shot 2022-08-31 at 1.47.48 PM.png (1×1 px, 646 KB)


  • Chat group links should only be displayed to participants after someone registers for event

Screen Recording 2022-08-31 at 1.49.49 PM.mov-04FA4881-6D31-4ACA-901E-0C2FAE020A87.gif (350×720 px, 723 KB)


  • The chat group information should be displayed in the same format/structure as it is displayed in EventDetails

@Daimona this structure is slightly different. Tagging @gonyeahialam for thoughts. See differences below.

In EventDetails the text is:
Chat group invite link

In more details modal the text is:
Join event chat group

The icons are also different and placed differently as seen below:

EventDetails page:

Screen Shot 2022-08-31 at 2.06.02 PM.png (988×1 px, 359 KB)

More details popup:

Screen Shot 2022-08-31 at 11.25.32 AM.png (1×1 px, 652 KB)


Also the Join event chat group link is underlined in the comp but it is not in the build. I assume this is fine because it is underlined on hover like other links in the extension, but just noting it here.

@Daimona this structure is slightly different.

Thanks for pointing this out. I mostly implemented what was in the prototype, so if that's different from what we have in Special:EventDetails, I'm guessing that the AC is wrong.

@Daimona this structure is slightly different.

Thanks for pointing this out. I mostly implemented what was in the prototype, so if that's different from what we have in Special:EventDetails, I'm guessing that the AC is wrong.

I thought that might be the case. I'll move this to design sign off to make sure, and then we can change the AC as needed.

The chat group information should be displayed in the same format/structure as it is displayed in EventDetails

@vaughnwalters we can remove the above as an acceptance criterion. I will update the design of EventDetails in T316606.

I now see the chat group link in 'more details' (see screenshot example below), so I'm marking this as Done.

Screen Shot 2022-09-20 at 10.17.22 AM.png (728×1 px, 78 KB)