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
F15415671: D7405.diff
Thu, Mar 20, 7:19 AM
F15385444: D7405.id.diff
Fri, Mar 14, 10:37 PM
F15347601: D7405.diff
Mon, Mar 10, 2:07 PM
Unknown Object (File)
Thu, Feb 27, 3:11 AM
Unknown Object (File)
Thu, Feb 27, 3:11 AM
Unknown Object (File)
Wed, Feb 26, 6:30 AM
Unknown Object (File)
Wed, Feb 26, 6:30 AM
Unknown Object (File)
Jan 27 2025, 7:07 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