Page MenuHomePhabricator

Community Configuration: Improve explanation for "underlinkedWeight" and "minimumLinkScore" in Suggested Edits form
Closed, ResolvedPublic

Description

User Story:

As an admin, I want to be able to adjust the "Add a link" task, because then I can ensure it works well for both newcomers and patrollers.

Details

Currently the on Suggested Edits editor form, under the "Add a link" task, there are two fields which are not configurable in Special:EditGrowthConfig:

Screenshot 2024-06-11 at 10.04.50 AM.png (326×964 px, 20 KB)

We should improve the copy and validation for these fields to ensure communities know how to use these fields and there is validation to ensure only acceptable values are added.


Previous work related to Weight of underlinked articles: T301096: Add a link: prioritize suggestions of underlinked articles / https://phabricator.wikimedia.org/T301096#8388234

Previous work related to the Minimum required link score:
T316079: Bump threshold for confidence score on link recommendation service suggestions


Acceptance Criteria

Update copy:
Old copy:

Weight of underlinked articles

New copy:

Communities can adjust how underlinked articles are prioritized. This field requires a percentage in its decimal form, so the number should be between 0 and 1. The more you increase this number, the more likely newcomers will receive underlinked article suggestions.

Add validation:
This field should only accept a number that is ≥0 and ≤ 1.


Old copy:

Minimum required link score

New copy:

Communities can adjust the confidence threshold for link recommendations. This field requires a percentage in its decimal form, so the number should be between 0 and 1. If you increase the number, the suggestions presented to the end user will have a higher likelihood of being good suggestions, however fewer suggestions will be available. If you decrease the number, there will be more suggestions available but some will have a lower likelihood of being good quality suggestions.

Add validation:
This field should only accept a number that is ≥0 and ≤ 1.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
KStoller-WMF renamed this task from Hide or remove "underlinkedWeight" and "minimumLinkScore" from Suggested Edits form to Community Configuration: Improve explanation for "underlinkedWeight" and "minimumLinkScore" in Suggested Edits form.Jun 11 2024, 5:46 PM
KStoller-WMF updated the task description. (Show Details)

Change #1046610 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] Improve explanation for underlinkedWeight / minimumLinkScore

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

Done. FWIW, I rephrased the beginning of both messages to not say "Communities can", as that is a phrasing that makes sense at eg. project pages (which address multiple communities at once), while the interface messages only address a single user, rather than multiple users (or communities) at the same time. This is how it looks with my patch applied:

image.png (1×1 px, 208 KB)

Hope this is fine :). Moving to Code Review!

Change #1046610 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Improve explanation for underlinkedWeight / minimumLinkScore

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

Done. FWIW, I rephrased the beginning of both messages to not say "Communities can", as that is a phrasing that makes sense at eg. project pages (which address multiple communities at once), while the interface messages only address a single user, rather than multiple users (or communities) at the same time. This is how it looks with my patch applied:

image.png (1×1 px, 208 KB)

Hope this is fine :). Moving to Code Review!

Checked in beta and testwiki - looks fine.
@Urbanecm_WMF - two questions
(1) I thought that the validation messages fall back to a browser default warning(s); in fact, it's what I see now in beta and on production.

Screen Shot 2024-06-26 at 7.26.48 PM.png (604×1 px, 128 KB)

(2) on wikis where CommunityConfiguratino is enabled, very often I see Minimum required link score field blank, e.g. https://es.wikipedia.org/wiki/Especial:Configuración_comunitaria/GrowthSuggestedEdits. Should the default value for this field be displayed?

[...]
Checked in beta and testwiki - looks fine.

Yay! :)

@Urbanecm_WMF - two questions
(1) I thought that the validation messages fall back to a browser default warnings; in fact, it's what I see now in beta and on production.

Screen Shot 2024-06-26 at 7.26.48 PM.png (604×1 px, 128 KB)

Indeed. I was testing what happens when I manually change the HTML via Developer tools, and remove the min/max requirements on the frontend. The screenshot illustrates that even in that case, the validation works correctly and as expected. One would achieve the same result by testing via the CommunityConfiguration API instead.

(2) on wikis where CommunityConfiguratino is enabled, very often I see Minimum required link score field blank, e.g. https://es.wikipedia.org/wiki/Especial:Configuración_comunitaria/GrowthSuggestedEdits. Should the default value for this field be displayed?

This is definitely an error and not intended, the default value should show. I'm not sure why it doesn't. @KStoller-WMF mentioned the same issue few days ago, and she said she'll fill a task for it, but I can't see any in CommunityConfiguration-Adoption or MediaWiki-extensions-CommunityConfiguration. I filled this as T368606: Community configuration defaults are not merged with partially-specified objects.

Thanks, @Urbanecm_WMF for the comments and clarification - closing this task as Resolved.