Page MenuHomePhabricator

Remove horrible old repository edit controller
ClosedPublic

Authored by epriestley on Oct 25 2013, 11:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 17 2026, 10:59 AM
Unknown Object (File)
Mar 8 2026, 7:14 PM
Unknown Object (File)
Mar 3 2026, 3:11 AM
Unknown Object (File)
Feb 6 2026, 7:55 PM
Unknown Object (File)
Jan 31 2026, 9:07 PM
Unknown Object (File)
Jan 31 2026, 2:35 PM
Unknown Object (File)
Jan 4 2026, 7:47 AM
Unknown Object (File)
Jan 4 2026, 7:28 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.