| MHorsey-WMF | |
| Jan 10 2023, 10:31 AM |
| F37731025: Screenshot 2023-09-18 at 9.03.35 PM.png | |
| Sep 19 2023, 6:01 AM |
| F37731023: Screenshot 2023-09-18 at 8.56.59 PM.png | |
| Sep 19 2023, 6:01 AM |
| F37731027: Screenshot 2023-09-18 at 9.05.09 PM.png | |
| Sep 19 2023, 6:01 AM |
| F37731002: Screenshot 2023-09-18 at 7.45.31 PM.png | |
| Sep 19 2023, 6:01 AM |
| F37730999: Screenshot 2023-09-18 at 7.39.34 PM.png | |
| Sep 19 2023, 6:01 AM |
| F37626913: Screen Recording 2023-08-24 at 10.33.15 AM.gif | |
| Aug 24 2023, 3:58 PM |
| F36916164: image.png | |
| Aug 24 2023, 3:58 PM |
| F37626866: Screenshot 2023-08-24 at 10.32.47 AM.png | |
| Aug 24 2023, 3:58 PM |
As an organiser, I want to be able to select participants to send emails to
And when I send an email it should be delivered only to those participants I have selected
As a participant,
When I have configured an email address on my account
And my account is not suppressed
And I have not blocked the organiser from sending me emails
I want to be able to receive emails from organisers
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add warning when organizer does not have a set email address. | mediawiki/extensions/CampaignEvents | master | +51 -11 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | MHorsey-WMF | T325098 [EPIC] Event organizer can email participants | |||
| Resolved | MHorsey-WMF | T326413 Implement front end module for participant email, compatible with current tab layout | |||
| Resolved | MHorsey-WMF | T326619 Implement participant selection logic and interface |
As an organiser, I want to be able to select participants to send emails to
✅ And when I send an email it should be delivered only to those participants I have selected
As a participant,
When I have configured an email address on my account
And my account is not suppressed ( I will need to test this on betacluster)
And I have not blocked the organiser from sending me emails
✅ I want to be able to receive emails from organisers
Send functionality works on local, but I will want to test this on betacluster as well.
One note @MHorsey-WMF, if an organizer does not have an email address, they are still able to click on the message participants button and it does not show a failure until the organizer clicks send on the messaging tab. Per this comment: T325098#8704990, the Message participants button should be disabled and have a tooltip if the organizer does not have an email address, and on the messaging tab send button should remain disabled and there should be an inline message warning the organizer that they need an email address to send a message.
One note @MHorsey-WMF, if an organizer does not have an email address, they are still able to click on the message participants button and it does not show a failure until the organizer clicks send on the messaging tab. Per this comment: T325098#8704990, the Message participants button should be disabled and have a tooltip if the organizer does not have an email address, and on the messaging tab send button should remain disabled and there should be an inline message warning the organizer that they need an email address to send a message.
Heya @MHorsey-WMF going back through this, on the participants tab, the message participants button is not disabled and the tooltip is not present if the organizer does not have an email address.
| comp | build |
 |  |
Also on the Message tab, There is no notice To message participants, you'll need an email address on your account. Add email.. Right now, it does not allow the organizer to add participants, but just sends them back to the participants tab.
| comp | build |
 |  |
Change 953983 had a related patch set uploaded (by Mhorsey; author: Mhorsey):
[mediawiki/extensions/CampaignEvents@master] Add warning when organizer does not have a set email address.
Change 953983 merged by jenkins-bot:
[mediawiki/extensions/CampaignEvents@master] Add warning when organizer does not have a set email address.
The AC in this task is met, but there are a few UI issues, and if these should go in another ticket I can make that instead:
| comp | build |
|  |
| comp | build |
|  |
 |  |
| comp | build |
 | |
The build is correct. This page was updated for the Participant questions feature/tasks.
- The Message participants button should be disabled (gray button) if the organizer does not have an email. The tool tip copy is correct, but right now it is on click of the button - @gonyeahialam can you clarify if this tooltip should appear on click or on hover of the disabled button?
comp build
If we leave the button disabled then the tooltip will only show on hover. However, we can leave it as it is in the build where the button isn't disabled and the tooltip shows on click.
- The To message participants... warning should go above the Recipients / Add recipients button
comp build
Yes, it is above. The warning is a central warning for the whole page.
- The Delete and Message participants buttons have a slight vertical alignment issue. When inspecting it you can see where it is off:
- The Delete button and the Message Participants button should not have any space between the buttons.
comp build
Yes, the buttons are together, the component is called a ButtonGroupWidget (destructive and progressive ButtonWidget)
Okay, great. 👍
- The Message participants button should be disabled (gray button) if the organizer does not have an email. The tool tip copy is correct, but right now it is on click of the button - @gonyeahialam can you clarify if this tooltip should appear on click or on hover of the disabled button?
comp build If we leave the button disabled then the tooltip will only show on hover. However, we can leave it as it is in the build where the button isn't disabled and the tooltip shows on click.
Okay thanks, leaving this as in then with no followup task.
- The To message participants... warning should go above the Recipients / Add recipients button
comp build Yes, it is above. The warning is a central warning for the whole page.
Created follow up task for this T346847
- The Delete and Message participants buttons have a slight vertical alignment issue. When inspecting it you can see where it is off:
- The Delete button and the Message Participants button should not have any space between the buttons.
comp build Yes, the buttons are together, the component is called a ButtonGroupWidget (destructive and progressive ButtonWidget)
Created followup task for this T346845
AC is met, created follow up tasks for UI issues, moving this to design sign off.
As the AC are met and we have a separate parent task for UX/UI changes, I am marking this work as Done.