@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Fix wording of "differential.sticky-accept"
Summary: See <http://fab.wmflabs.org/T226>. The summary for this option is confusing, because "true" means sticky but the wording implies "true" means non-sticky.
Test Plan:
- Looked at the option in summary view.
- Reviewed related text, none of the other copy here seems confusing or ambiugous to me.
Reviewers: btrahan, chad
Reviewed By: chad
Subscribers: qgil, epriestley
Differential Revision: https://secure.phabricator.com/D8884
+1
-1
+1
-1
src/applications/differential/config/PhabricatorDifferentialConfigOptions.php
+1
-1
src/applications/differential/config/PhabricatorDifferentialConfigOptions.php
···
124
124
pht("Accepts are reset by updates"),
125
125
))
126
126
->setSummary(
127
-
pht("Should updating an accepted revision require re-review?"))
127
+
pht('Should "Accepted" revisions remain "Accepted" after updates?'))
128
128
->setDescription(
129
129
pht(
130
130
'Normally, when revisions that have been "Accepted" are updated, '.