Page MenuHomePhabricator
Differential D15883

Restore edit UI for "Import Only" in Subversion
ClosedPublic
Actions

Authored by epriestley on May 11 2016, 12:14 AM.
Tags
None
Referenced Files
F13233774: D15883.diff
Tue, May 21, 2:34 AM
F13224739: D15883.id38263.diff
Sun, May 19, 10:32 AM
F13211992: D15883.diff
Fri, May 17, 6:18 AM
F13206530: D15883.diff
Wed, May 15, 11:42 AM
F13206024: D15883.id38263.diff
Wed, May 15, 4:45 AM
F13199384: D15883.id38265.diff
Mon, May 13, 2:15 PM
F13195631: D15883.diff
Sun, May 12, 10:31 PM
F13190668: D15883.id38251.diff
Sat, May 11, 1:07 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10923: Diffusion API/URI Errata
Commits
rP97c103fa00cc: Restore edit UI for "Import Only" in Subversion
Summary

Ref T10923. Although I'd ideally like to get rid of this eventually, keep it around for now.

Test Plan
  • Edited value for an SVN repository.
  • Observed no panel present for a Git repository.

Diff Detail

Repository
rP Phabricator
Branch
dxdoc13
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12138
Build 15311: Run Core Tests
Build 15310: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 38251.May 11 2016, 12:14 AM
epriestley retitled this revision from to Restore edit UI for "Import Only" in Subversion.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10923: Diffusion API/URI Errata.
chad accepted this revision.May 11 2016, 3:07 AM
chad edited edge metadata.

this also has bad unit tests? not related?

This revision is now accepted and ready to land.May 11 2016, 3:07 AM
epriestley updated this revision to Diff 38263.May 11 2016, 1:39 PM
epriestley edited edge metadata.
  • Rebase for test fix.
Closed by commit rP97c103fa00cc: Restore edit UI for "Import Only" in Subversion (authored by epriestley, committed by epriestley). · Explain WhyMay 11 2016, 1:47 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in Z1336: General Chat.May 11 2016, 4:20 PM

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
diffusion/
editor/
16 lines
management/
77 lines

Diff 38263

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php

Loading...
View Options

src/applications/diffusion/management/DiffusionRepositorySubversionManagementPanel.php

Loading...
Phabricator · Built by Phacility