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

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

Tags

None

Referenced Files

	F14043858: D8884.diff
	Tue, Nov 12, 1:26 PM

	F14039000: D8884.id.diff
	Mon, Nov 11, 3:08 AM

	F14038139: D8884.id21077.diff
	Sun, Nov 10, 9:46 PM

	F14028152: D8884.diff
	Fri, Nov 8, 12:01 PM

	F14017035: D8884.id21077.diff
	Mon, Nov 4, 1:50 PM

	F14012832: D8884.id21073.diff
	Fri, Nov 1, 7:44 PM

	F14003308: D8884.diff
	Sat, Oct 26, 4:35 AM

	F13992545: D8884.id21077.diff
	Tue, Oct 22, 5:16 PM

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

Branch

hword1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 99
Build 99: [Placeholder Plan] Wait for 30 Seconds

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 21073

View Options

src/applications/differential/config/PhabricatorDifferentialConfigOptions.php

Fix wording of "differential.sticky-accept"ClosedPublicActions