Page MenuHomePhabricator
Differential D7813

Herald - add support for "assignee" conditions
ClosedPublic
Actions

Authored by xiaogaozi on Dec 21 2013, 3:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 22, 11:36 AM
Unknown Object (File)
Tue, Oct 6, 12:47 AM
Unknown Object (File)
Fri, Oct 2, 7:30 AM
Unknown Object (File)
Thu, Oct 1, 8:57 PM
Unknown Object (File)
Tue, Sep 29, 1:17 PM
Unknown Object (File)
Sep 18 2026, 12:43 AM
Unknown Object (File)
Sep 5 2026, 3:38 AM
Unknown Object (File)
Aug 20 2026, 12:54 PM
View All Files
Subscribers
aran
epriestley
Korvin
Tokens
"Piece of Eight" token, awarded by Leenug.

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
Restricted Diffusion Commit
Summary

add support for "assignee" conditions

Test Plan

Create a Herald rule where condition is assignee, and create a task assign to someone.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley accepted this revision.Dec 22 2013, 4:47 PM

Thanks! I'm going to make one minor change in the pull (see inline) to make this slightly more flexible.

src/applications/herald/adapter/HeraldAdapter.php
234–235

I think this field should use this set of conditions instead, because a task may not be assigned, so "Assignee does not exist" is a meaningful filter for finding unowned tasks.

epriestley closed this revision.Dec 22 2013, 4:48 PM

Closed by commit rP54a0dd813908 (authored by @xiaogaozi, committed by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
herald/
adapter/
3 lines
3 lines
storage/
2 lines

Diff 17689

View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldManiphestTaskAdapter.php

Loading...
View Options

src/applications/herald/storage/HeraldRule.php

Loading...
Phabricator · Built by Phacility