Page MenuHomePhabricator

allow phids other than users (i.e. mailinglists)
ClosedPublic

Authored by chasemp on Aug 8 2014, 4:52 PM.
Tags
None
Referenced Files
F15400791: D10193.id.diff
Mon, Mar 17, 2:01 PM
F15400325: D10193.id24520.diff
Mon, Mar 17, 11:01 AM
F15395812: D10193.diff
Sun, Mar 16, 9:54 AM
F15386290: D10193.id24520.diff
Sat, Mar 15, 12:18 AM
F15334781: D10193.diff
Sat, Mar 8, 12:05 PM
Unknown Object (File)
Mon, Mar 3, 8:52 AM
Unknown Object (File)
Wed, Feb 26, 7:04 PM
Unknown Object (File)
Feb 23 2025, 2:42 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP57eb3aecb68f: allow phids other than users (i.e. mailinglists)
Summary

Via the UI adding a mailinglist for CC works, but via
the API currently it shows:

One or more PHIDs were invalid for ccPHIDS

This removes the user validation check for ccPHIDs.

(I left it in for other things like owner since that seems
still appropriate?)

Test Plan

used arc locally to add a mailinglist to cc

echo '{"id": 2, "ccPHIDs": ["PHID-MLST-ohduchbv4dfimk7opt3r"]}' | arc call-conduit maniphest.update

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

chasemp retitled this revision from to allow phids other than users (i.e. mailinglists).
chasemp updated this object.
chasemp edited the test plan for this revision. (Show Details)
chasemp added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 8 2014, 4:53 PM
epriestley updated this revision to Diff 24521.

Closed by commit rP57eb3aecb68f (authored by cpettet <rush@wikimedia.org>, committed by @epriestley).