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)
Thu, Jan 2, 4:07 PM
Unknown Object (File)
Fri, Dec 27, 3:13 PM
Unknown Object (File)
Dec 27 2024, 12:58 AM
Unknown Object (File)
Dec 27 2024, 12:55 AM
Unknown Object (File)
Dec 27 2024, 12:55 AM
Unknown Object (File)
Dec 27 2024, 12:43 AM
Unknown Object (File)
Dec 25 2024, 2:51 AM
Unknown Object (File)
Dec 20 2024, 11:52 PM
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