Fix wording of "differential.sticky-accept"
ClosedPublic
Actions

Authored by epriestley on Apr 28 2014, 4:26 PM.

Tags

None

Referenced Files

	F15470220: D8884.id21073.diff
	Fri, Apr 4, 9:19 PM

	F15468699: D8884.diff
	Fri, Apr 4, 4:36 AM

	F15430919: D8884.diff
	Mon, Mar 24, 9:39 AM

	F15425407: D8884.id21073.diff
	Sun, Mar 23, 4:36 AM

	F15410448: D8884.id21073.diff
	Wed, Mar 19, 7:25 AM

	F15393269: D8884.id21073.diff
	Sat, Mar 15, 8:46 PM

	F15382355: D8884.diff
	Fri, Mar 14, 12:00 PM

	F15376160: D8884.id21073.diff
	Thu, Mar 13, 2:11 AM

View All Files

Subscribers

epriestley

qgil

Tokens

"Like" token, awarded by qgil.

Details

Reviewers

btrahan
chad

Commits

Restricted Diffusion Commit
rPbf6bda6ef481: 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.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 21073.Apr 28 2014, 4:26 PM

epriestley retitled this revision from to Fix wording of "differential.sticky-accept".

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: btrahan, chad.

epriestley added a subscriber: qgil.

Herald added a subscriber: epriestley. · View Herald TranscriptApr 28 2014, 4:26 PM

Harbormaster completed remote builds in B99: Diff 21073.Apr 28 2014, 4:27 PM

chad accepted this revision.Apr 28 2014, 4:27 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Apr 28 2014, 4:27 PM

Closed by commit rPbf6bda6ef481 (authored by @epriestley).

qgil awarded a token.Apr 28 2014, 4:38 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

config/

PhabricatorDifferentialConfigOptions.php

2 lines

Diff 21077

View Options

src/applications/differential/config/PhabricatorDifferentialConfigOptions.php

Fix wording of "differential.sticky-accept"ClosedPublicActions