Page MenuHomePhabricator

Remove horrible old repository edit controller
ClosedPublic

Authored by epriestley on Oct 25 2013, 11:25 PM.
Tags
None
Referenced Files
F19015128: D7410.diff
Sat, Nov 22, 11:18 PM
F18869073: D7410.id.diff
Tue, Nov 4, 9:49 AM
F18864431: D7410.diff
Mon, Nov 3, 5:36 AM
F18728961: D7410.id.diff
Sep 30 2025, 10:15 AM
F18713207: D7410.diff
Sep 29 2025, 5:43 AM
F18579366: D7410.diff
Sep 11 2025, 12:31 AM
F17851983: D7410.id.diff
Jul 27 2025, 12:01 PM
F17828106: D7410.diff
Jul 26 2025, 8:52 AM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPc585f97e90d2: Remove horrible old repository edit controller
Summary

Ref T2231. I didn't port these options over, so they're still supported but have no edit UI:

  • Pull Frequency (confusing/not useful, I think?)
  • Default Owners Path (probably used only by Facebook and only in the E repository)
  • Show user in public repository URL (probably mostly obsolete with hosting?)

We can add those back if users notice, but they seem like the three least useful options so I'm going to see if we can get away with removing them.

Test Plan

Clicked "Edit" from Repositories, got kicked into the nice new Diffusion edit UI instead of the old one.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Cool. I like the strat here too.