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
F13295651: D15303.id36912.diff
Thu, Jun 6, 6:30 AM
F13291648: D15303.diff
Wed, Jun 5, 12:08 AM
F13268261: D15303.diff
Wed, May 29, 4:51 AM
F13256655: D15303.diff
Sat, May 25, 3:42 PM
F13233307: D15303.diff
Tue, May 21, 2:12 AM
F13211311: D15303.diff
Fri, May 17, 5:39 AM
F13196568: D15303.diff
Sun, May 12, 11:22 PM
F13176531: D15303.id36922.diff
May 8 2024, 2:48 PM
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