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
Unknown Object (File)
Tue, Mar 26, 3:09 PM
Unknown Object (File)
Mon, Mar 4, 6:01 PM
Unknown Object (File)
Feb 17 2024, 10:14 AM
Unknown Object (File)
Feb 17 2024, 10:14 AM
Unknown Object (File)
Feb 17 2024, 10:14 AM
Unknown Object (File)
Feb 17 2024, 4:40 AM
Unknown Object (File)
Feb 11 2024, 3:47 PM
Unknown Object (File)
Feb 11 2024, 4:52 AM
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
Branch
maniphest_update_mailinglist_ccphids
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2109
Build 2113: [Placeholder Plan] Wait for 30 Seconds

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).