For the last weeks we've seen an influx of activity by new technical contributors in Phabricator tasks outside of the outreach program patterns.
The communication style seems to also become more AI-guided (personal impression).
So I've wondered again how to better support folks to better/faster (?) get along in this Wikimedia environment.
For the records, we've put some stuff in place in the past. Which we have never evaluated afaik:
- T369982: Display info for new technical contributors in task comment field for new accounts (only displayed on good first task s but... that requires awareness that good first task exists)
- T367500: Explore displaying additional info in comment field for open Outreach related tasks (requires regular work twice a year, which I have not performed)
- rPHABea784458932e: Show info for new contributors in task comment field (covering both tasks)
- T330379: Set up Herald rule to automatically answer some common newcomer questions in {H414} (disabled because too heavy)
Another rough idea (but that ignores what is already in place):
diff --git a/src/applications/transactions/view/PhabricatorApplicationTransactionCommentView.php b/src/applications/transactions/view/PhabricatorApplicationTransactionCommentView.php index 4415199172..8fdd3dafa7 100644 --- a/src/applications/transactions/view/PhabricatorApplicationTransactionCommentView.php +++ b/src/applications/transactions/view/PhabricatorApplicationTransactionCommentView.php @@ -325,6 +325,25 @@ final class PhabricatorApplicationTransactionCommentView } else { $placeholder_text = ''; } +// WMF BEGIN T433135 + $newcomer_comment_caption = null; + $now = time(); + $viewer_created = $viewer->getDateCreated(); + $viewer_age = $now - $viewer_created; + if ($viewer_age < (60*60*24*31) && $engine instanceof ManiphestEditEngine) { + $new_developers_uri = 'https://www.mediawiki.org/wiki/New_Developers#Communication_tips'; + $new_developers_link = pht('Find out what to do instead'); + $commit_guidelines_uri = 'https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines'; + $commit_guidelines_link = pht('Commit Message Guidelines'); + $newcomer_comment_caption = pht( + 'Please do not add a comment to announce that you claim or work on this '. + 'task: %s. Please do not add a comment to announce that you '. + 'submitted a patch: Follow the %s to create an automatic notification '. + 'about your patch in this task instead. Thank you!', + phutil_tag('a', array('href' => $new_developers_uri), $new_developers_link), + phutil_tag('a', array('href' => $commit_guidelines_uri), $commit_guidelines_link)); + } +// WMF END T433135 $remarkup_control = id(new PhabricatorRemarkupControl()) ->setViewer($viewer) @@ -332,6 +351,7 @@ final class PhabricatorApplicationTransactionCommentView ->addClass('phui-comment-fullwidth-control') ->addClass('phui-comment-textarea-control') ->setSurroundingObject($this->getObject()) + ->setCaption($newcomer_comment_caption) // WMF T433135 ->setCanPin(true) ->setPlaceholder($placeholder_text) ->setName('comment');
