Page MenuHomePhabricator

Update redirect/cancel URIs for repository dialogs
ClosedPublic

Authored by epriestley on May 9 2016, 2:07 PM.
Tags
None
Referenced Files
F13261928: D15867.diff
Mon, May 27, 1:36 AM
F13258303: D15867.id.diff
Sun, May 26, 8:12 AM
F13187065: D15867.diff
Sat, May 11, 4:18 AM
F13180203: D15867.id38221.diff
Wed, May 8, 10:37 PM
F13174722: D15867.diff
Wed, May 8, 12:50 AM
Unknown Object (File)
Tue, May 7, 7:23 AM
Unknown Object (File)
May 5 2024, 9:47 AM
Unknown Object (File)
May 3 2024, 6:07 AM
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.