Page MenuHomePhabricator

Application Emails - add datasource so we can have a typeahead
ClosedPublic

Authored by btrahan on Jan 28 2015, 10:10 PM.
Tags
None
Referenced Files
F14347902: D11546.diff
Thu, Dec 19, 3:54 AM
F14344094: D11546.id27796.diff
Thu, Dec 19, 12:32 AM
Unknown Object (File)
Mon, Dec 16, 11:16 PM
Unknown Object (File)
Thu, Dec 5, 12:40 PM
Unknown Object (File)
Wed, Nov 27, 3:02 PM
Unknown Object (File)
Fri, Nov 22, 11:39 PM
Unknown Object (File)
Nov 19 2024, 3:41 AM
Unknown Object (File)
Nov 15 2024, 1:48 PM
Subscribers

Details

Summary

Ref T5039. This will be necessary for Herald integration so users can make rules like "if app email is one of x, y, or z add projects foo, bar, and metallica." I think its best to do an actual typeahead here -- users select full email addresses -- rather than support prefix, suffix, etc stuff on the email address. I think the latter approach would yield lots of confusion, as well as prevent us from (more) easily providing diagnostic tools about what happened when and why.

Test Plan

hacked a maniphest tokenizer to use this new datasource and it worked

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Application Emails - add datasource so we can have a typeahead.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

Ah, yeah, this is probably a better approach.

One maybe-weird thing is that editing an email will affect rules, but that's probably reasonable/expected.

This revision is now accepted and ready to land.Jan 28 2015, 10:14 PM
This revision was automatically updated to reflect the committed changes.