Differential D20448

Don't warn that workboard columns need a name when editing milestone columns
ClosedPublic
Actions

Authored by epriestley on Apr 18 2019, 12:31 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jun 25, 6:10 PM

	Unknown Object (File)
	Tue, Jun 24, 2:58 PM

	Unknown Object (File)
	Thu, Jun 5, 4:20 PM

	Unknown Object (File)
	May 25 2025, 9:27 PM

	Unknown Object (File)
	May 7 2025, 12:53 AM

	Unknown Object (File)
	May 1 2025, 11:46 PM

	Unknown Object (File)
	Apr 24 2025, 10:03 PM

	Unknown Object (File)
	Apr 6 2025, 7:46 PM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Commits

rPcd2215fd4a69: Don't warn that workboard columns need a name when editing milestone columns

Summary

See https://discourse.phabricator-community.org/t/columns-must-have-a-name-while-editong-point-limit-on-milestone-column/2650. This check doesn't make sense for proxy columns, including milestone columns.

Test Plan

Added a point limit to a milestone column.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 18 2019, 12:31 PM

Harbormaster completed remote builds in B22654: Diff 48782.Apr 18 2019, 12:33 PM

epriestley requested review of this revision.Apr 18 2019, 12:33 PM

amckinley accepted this revision.Apr 18 2019, 4:40 PM

This revision is now accepted and ready to land.Apr 18 2019, 4:40 PM

Closed by commit rPcd2215fd4a69: Don't warn that workboard columns need a name when editing milestone columns (authored by epriestley). · Explain WhyApr 18 2019, 8:13 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

xaction/

column/

PhabricatorProjectColumnNameTransaction.php

5 lines

Diff 48800

View Options

src/applications/project/xaction/column/PhabricatorProjectColumnNameTransaction.php

Don't warn that workboard columns need a name when editing milestone columnsClosedPublicActions