Differential D20946

Add "Project tags added" and "Project tags removed" fields in Herald
ClosedPublic
Actions

Authored by epriestley on Jan 21 2020, 7:31 PM.

Tags

None

Referenced Files

	F15580641: D20946.diff
	Wed, May 7, 6:55 AM

	F15532947: D20946.diff
	Wed, Apr 23, 5:43 PM

	F15523333: D20946.diff
	Mon, Apr 21, 12:41 AM

	F15516367: D20946.id.diff
	Fri, Apr 18, 6:42 PM

	F15512907: D20946.diff
	Thu, Apr 17, 2:49 PM

	F15502114: D20946.diff
	Sun, Apr 13, 10:43 PM

	F15446200: D20946.id.diff
	Mar 27 2025, 5:42 PM

	F15424160: D20946.id.diff
	Mar 22 2025, 8:36 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13480: Implement "Author's packages", "Added project tags", and "Removed project tags" Herald fields
Commits: rP6c4500046f7c: Add "Project tags added" and "Project tags removed" fields in Herald

Summary

Ref T13480. These fields don't serve a specific strong use case, but are broadly reasonable capabilities after "state" vs "change" actions were relaxed by T13283.

Test Plan

Wrote rules using the new fields, added and removed projects (and did neither) to fire them / not fire them. Inspected the transcripts to see the project PHIDs making it to the field values.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 21 2020, 7:31 PM

Harbormaster completed remote builds in B23733: Diff 49907.Jan 21 2020, 7:33 PM

epriestley requested review of this revision.Jan 21 2020, 7:33 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jan 23 2020, 2:21 AM

Closed by commit rP6c4500046f7c: Add "Project tags added" and "Project tags removed" fields in Herald (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP6c4500046f7c: Add "Project tags added" and "Project tags removed" fields in Herald.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

applications/

herald/

field/

HeraldField.php

45 lines

project/

herald/

HeraldProjectsField.php

19 lines

PhabricatorProjectTagsAddedField.php

23 lines

	PhabricatorProjectTagsField.php
	HeraldProjectsField.php

20 lines

PhabricatorProjectTagsRemovedField.php

23 lines

Diff 49909

src/__phutil_library_map__.php

Loading...

src/applications/herald/field/HeraldField.php

Loading...

src/applications/project/herald/HeraldProjectsField.php

Loading...

src/applications/project/herald/PhabricatorProjectTagsAddedField.php

Loading...

src/applications/project/herald/PhabricatorProjectTagsField.php

Loading...

src/applications/project/herald/PhabricatorProjectTagsRemovedField.php

Loading...