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
F18437058: D7813.id.diff
Sun, Aug 31, 11:44 AM
F18435444: D7813.id17689.diff
Sun, Aug 31, 10:06 AM
F18417489: D7813.diff
Sat, Aug 30, 11:53 AM
F18368364: D7813.id17680.diff
Thu, Aug 28, 12:49 AM
F18363421: D7813.id17680.diff
Wed, Aug 27, 4:31 PM
F18362778: D7813.id17689.diff
Wed, Aug 27, 3:00 PM
F18362393: D7813.diff
Wed, Aug 27, 1:45 PM
F18038395: D7813.id17689.diff
Aug 3 2025, 2:58 AM
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
rP54a0dd813908: Herald - add support for "assignee" conditions
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