Page MenuHomePhabricator

Allow Conduit API methods in Diffusion to accept any repository identifier
ClosedPublic

Authored by epriestley on Jan 2 2016, 7:20 PM.
Tags
None
Referenced Files
F15568123: D14928.id36080.diff
Fri, May 2, 8:33 PM
F15558125: D14928.id36070.diff
Tue, Apr 29, 1:11 AM
F15532738: D14928.id36080.diff
Wed, Apr 23, 5:06 PM
F15477561: D14928.id36071.diff
Apr 7 2025, 4:14 PM
F15477500: D14928.id36080.diff
Apr 7 2025, 3:28 PM
F15475587: D14928.id36070.diff
Apr 6 2025, 9:03 PM
F15475068: D14928.id.diff
Apr 6 2025, 3:19 PM
F15472592: D14928.diff
Apr 5 2025, 2:52 PM
Subscribers
None

Details

Summary

Ref T4245. Broaden support to include "ABCD", "rABCD", "1234", "R1234", etc.

This doesn't change the old behavior, just accepts more stuff.

Test Plan
  • Browsed Diffusion.
  • Made various calls via API console.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Allow Conduit API methods in Diffusion to accept any repository identifier.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
  • Fix an issue with juggling reposiories and identifiers.
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 3 2016, 3:59 AM
This revision was automatically updated to reflect the committed changes.