| cmelo | |
| Apr 13 2023, 1:25 PM |
| F36964022: Screenshot 2023-04-26 at 8.36.23 PM.png | |
| Apr 27 2023, 1:39 AM |
| F36962195: Screen Recording 2023-04-24 at 3.18.01 PM.gif | |
| Apr 24 2023, 8:42 PM |
| F36962163: Screen Recording 2023-04-24 at 2.36.38 PM.gif | |
| Apr 24 2023, 8:42 PM |
| F36962169: Screenshot 2023-04-24 at 2.47.54 PM.png | |
| Apr 24 2023, 8:42 PM |
Add the name of the event creator in case an organizer tries to remove the event creator
There is an error on OOUI that allows users to remove the event creator even when the tag is disabled.
We will add the username of the event creator, so in case the user remove the event creator by mistake (because of the OOUI error), the user knows who is the creator and can add it again to be able to edit the event registration
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add the username of the event creator in the error message | mediawiki/extensions/CampaignEvents | master | +28 -16 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | cmelo | T325101 [EPIC] Multiple organizer support | |||
| Resolved | cmelo | T334670 Add the name of the event creator in the error message in case an organizer tries to remove the event creator |
Change 909000 had a related patch set uploaded (by Cmelo; author: Cmelo):
[mediawiki/extensions/CampaignEvents@master] Add the username of the event creator in the error message
@gonyeahialam I would like your thoughts on how the error message would look like, now that we will add the username of the event creator, for now I put it this way:
Before: You are not allowed to remove the event creator
Now: You are not allowed to remove the event creator (UserName)
Use this error message @cmelo:
"You cannot remove (UserName) because they are the event creator"
Just registering here, what we (me and @gonyeahialam) decide on slack, that message will be:
"You cannot remove (UserName) because this user is the event creator"
Thanks @gonyeahialam!
Noting that
conflicts with
Which one should we use? Also, if choosing the former, we should use the appropriate pronoun with {{GENDER}}, instead of unconditionally using "they".
We will use this one:
"You cannot remove (UserName) because this user is the event creator"
Change 909000 merged by jenkins-bot:
[mediawiki/extensions/CampaignEvents@master] Add the username of the event creator in the error message
Event Creator username successfully added to error message in case an organizer tries to remove the event creator on english betacluster
@cmelo In testing this on arabic betacluster though it does not display the username in the error message.
@cmelo why does the error message show at the top instead of below the field with the error?
It is because this one happens after you submit the form, but this is an edge case that only happens because of a bug on OOUI
Username (without parenthesis) is displaying in error message. Sending to design sign off.