Improve wording in TWL portal interface (T430924)#1539
Conversation
|
Hi @theresnotime , i noticed the cicd check failed again on test_merge_partner_suggestion_view_post after your re-run. this test checks suggestion merge upvote counts, which seems unrelated to this PR's wording-only changes. is this a known/pre-existing flaky test, or could it be related to something else? Happy to investigate further if needed. |
I'm not sure if this is a known flaky test, but I've re-run the CI and we'll see if it fails again :) |
|
Passed on a re-run - I've logged this as a potentially flaky test at https://phabricator.wikimedia.org/T431407. Thank you! |
|
@theresnotime thanks for confirming and logging that! since all checks are passing now, is there anything else needed from my side, or is this ready to be reviewed/merged? |
I've added it to our board for code review requests - someone should take a look soon :) |
|
Closing in favour of the existing (and updated) #1537 |
Description
Improves several small text labels on the TWL portal interface based on user feedback in the linked ticket:
Rationale
These changes were suggested by a user in the linked Phabricator ticket, who noted that several small text labels on the portal were unclear or ambiguous. The new wording aims to make these labels more self-explanatory for both new and existing users.
Phabricator Ticket
https://phabricator.wikimedia.org/T430924
How Has This Been Tested?
This is a text-only change. Each string was located in the codebase and verified against the surrounding context (filters, templates, email notifications) to confirm it matches the intended display location. No functional/logic changes were made.
Screenshots of your changes (if appropriate):
N/A - text/wording only changes
Types of changes
What types of changes does your code introduce? Add an
xin all the boxes that apply: