Page MenuHomePhabricator

Update PhabricatorRepositoryManagementLookupUsersWorkflow to use ConduitCall
ClosedPublic

Authored by epriestley on Sep 2 2014, 10:52 AM.
Tags
None
Referenced Files
F14837306: D10403.id25027.diff
Fri, Jan 31, 8:11 AM
F14836725: D10403.diff
Fri, Jan 31, 4:17 AM
Unknown Object (File)
Mon, Jan 27, 10:29 PM
Unknown Object (File)
Thu, Jan 23, 4:11 PM
Unknown Object (File)
Thu, Jan 23, 4:11 PM
Unknown Object (File)
Thu, Jan 23, 4:11 PM
Unknown Object (File)
Thu, Jan 23, 4:10 PM
Unknown Object (File)
Thu, Jan 23, 4:10 PM

Details

Summary

Ref T2783.

This updates PhabricatorRepositoryManagementLookupUsersWorkflow to use ConduitCall to retrieve information about the commit.

Test Plan

Ran bin/repository lookup-users rTESTe9683b64d3283f0b2d355fdbf231bc918b5ac0ab --trace and saw the information returned (by making a request to diffusion.querycommits as the omnipotent user, signed with the device key).

Mucked with cluster.addresses and saw requests rejected.

Event Timeline

hach-que retitled this revision from to Update PhabricatorRepositoryManagementLookupUsersWorkflow to use ConduitCall.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que edited edge metadata.

Use idx() to prevent PHP warnings

epriestley edited reviewers, added: hach-que; removed: epriestley.

dis mine now

  • Rebase on top of the various changes which have happened on this pathway.

I'll update the metadata in a sec..

btrahan edited edge metadata.
This revision is now accepted and ready to land.Jan 2 2015, 9:31 PM
This revision was automatically updated to reflect the committed changes.