Page MenuHomePhabricator

Move "Remote URI" to modern repository editor thing
ClosedPublic

Authored by epriestley on Oct 25 2013, 8:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 11:52 PM
Unknown Object (File)
Thu, Dec 19, 5:01 PM
Unknown Object (File)
Tue, Dec 17, 9:45 PM
Unknown Object (File)
Fri, Dec 13, 11:18 PM
Unknown Object (File)
Thu, Dec 12, 2:51 PM
Unknown Object (File)
Mon, Dec 9, 4:21 AM
Unknown Object (File)
Mon, Dec 9, 4:21 AM
Unknown Object (File)
Sun, Dec 8, 7:28 AM
Subscribers

Details

Reviewers
btrahan
chad
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPb57b72368c3c: Move "Remote URI" to modern repository editor thing
Summary

Ref T2231. Allows you to edit the remote URI and credentials.

This is a little bit funky because I'm reusing some of the pages on the new (not-yet-hooked-up) create form. Specifically, it had pages like this:

  • Repo Type
  • Name/Callsign/Remote
  • Auth
  • Done

I split "Name/Callsign/Remote" into "Name/Callsign" and "Remote", then when editing the remote I just take you through "Remote" and "Auth" and then back. This lets us reuse the giant pile of protocol/URI sanity checking logic and ends up being pretty clean, although it's a little weird that the "Create" controller does both full-create and edit-remote.

Test Plan

See screenshots.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped