Page MenuHomePhabricator
Differential D15303

Don't require a callsign to set a repository's local path
ClosedPublic
Actions

Authored by epriestley on Feb 18 2016, 1:29 PM.
Tags
None
Referenced Files
F18108956: D15303.id36912.diff
Mon, Aug 11, 10:44 AM
F18105518: D15303.id36912.diff
Sun, Aug 10, 6:26 PM
F18105517: D15303.id.diff
Sun, Aug 10, 6:26 PM
F18105516: D15303.id36922.diff
Sun, Aug 10, 6:26 PM
F18105401: D15303.diff
Sun, Aug 10, 5:53 PM
F18090747: D15303.id36922.diff
Wed, Aug 6, 5:15 PM
F18085798: D15303.id.diff
Tue, Aug 5, 7:25 PM
F17867441: D15303.id.diff
Mon, Jul 28, 6:09 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T4245: Make repository URIs slightly prettier
Commits
Restricted Diffusion Commit
rPb63eb09cacf1: Don't require a callsign to set a repository's local path
Summary

Ref T4245. When creating new repositories, set a default local path based on the repository ID instead of callsign.

Test Plan
  • Created a new repository.
  • Saw it get a reasonable, ID-based local path.
  • Edited a repository to make sure the applyFinalEffects() wasn't doing anything whacky.

Diff Detail

Repository
rP Phabricator
Branch
call9
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10755
Build 13238: Run Core Tests
Build 13237: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 36912.Feb 18 2016, 1:29 PM
epriestley retitled this revision from to Don't require a callsign to set a repository's local path.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T4245: Make repository URIs slightly prettier.
chad accepted this revision.Feb 18 2016, 4:16 PM
chad edited edge metadata.

6dd.png (600×600 px, 94 KB)

This revision is now accepted and ready to land.Feb 18 2016, 4:16 PM
Closed by commit rPb63eb09cacf1: Don't require a callsign to set a repository's local path (authored by epriestley, committed by epriestley). · Explain WhyFeb 18 2016, 6:33 PM
This revision was automatically updated to reflect the committed changes.
• Persona added a child revision: D15341: Fix Ponder Exception, spacing.Feb 24 2016, 11:16 AM
tycho.tatitscheff removed a child revision: D15341: Fix Ponder Exception, spacing.Feb 24 2016, 12:31 PM

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
controller/
11 lines
repository/
editor/
23 lines

Diff 36912

View Options

src/applications/diffusion/controller/DiffusionRepositoryCreateController.php

Loading...
View Options

src/applications/repository/editor/PhabricatorRepositoryEditor.php

Loading...
Phabricator · Built by Phacility