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
Unknown Object (File)
Feb 8 2024, 4:58 AM
Unknown Object (File)
Jan 29 2024, 6:05 AM
Unknown Object (File)
Dec 30 2023, 5:13 PM
Unknown Object (File)
Dec 26 2023, 6:30 PM
Unknown Object (File)
Dec 22 2023, 9:57 PM
Unknown Object (File)
Nov 25 2023, 2:06 PM
Unknown Object (File)
Nov 18 2023, 6:57 PM
Unknown Object (File)
Nov 16 2023, 7:24 AM
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.