Page MenuHomePhabricator

Update redirect/cancel URIs for repository dialogs
ClosedPublic

Authored by epriestley on May 9 2016, 2:07 PM.
Tags
None
Referenced Files
F18864320: D15867.id.diff
Mon, Nov 3, 4:38 AM
F18813063: D15867.diff
Oct 20 2025, 1:46 PM
F18784006: D15867.id.diff
Oct 13 2025, 11:34 AM
F18733339: D15867.id.diff
Sep 30 2025, 10:32 PM
F18714645: D15867.id38221.diff
Sep 29 2025, 8:34 AM
F18625488: D15867.diff
Sep 15 2025, 10:54 PM
F18424796: D15867.id.diff
Aug 30 2025, 9:21 PM
F18423785: D15867.id38221.diff
Aug 30 2025, 8:04 PM
Subscribers
None

Details

Summary

Ref T10923. Some of the dialogs ("Deactivate Repository", "Test Automation", etc.) had cancel or redirect URIs which I missed originally.

Go through them and make sure they all point to the right places.

Also removed one unused controller which I missed the first time around.

Test Plan
  • Opened all these dialogs in a new tab with Command-Click.
  • Clicked every "cancel" and "submit" button on all of these dialogs.
  • Got consistently sent to the place I came from.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Update redirect/cancel URIs for repository dialogs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.May 9 2016, 3:03 PM
This revision was automatically updated to reflect the committed changes.