Differential D16858

Don't show "clone-name" as "Short Name"
ClosedPublic
Actions

Authored by avivey on Nov 14 2016, 10:43 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rPd5a72ca98ed2: Don't show "clone-name" as "Short Name"

Summary

See D16851 - there's now a difference in their meaning, so don't unite them in the UI.

Test Plan

Load manage page of repos

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 14485
Build 18884: Run Core Tests
Build 18883: arc lint + arc unit

Event Timeline

avivey updated this revision to Diff 40594.Nov 14 2016, 10:43 PM

avivey retitled this revision from to Don't show "clone-name" as "Short Name".

avivey updated this object.

avivey edited the test plan for this revision. (Show Details)

avivey added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptNov 14 2016, 10:43 PM

Herald added a subscriber: Korvin. · View Herald Transcript

Maybe "No Short Name" or "No short name" or "None" (i.e., more consistent capitalization)?

This revision is now accepted and ready to land.Nov 14 2016, 10:44 PM

avivey updated this object.Nov 14 2016, 10:45 PM

avivey edited edge metadata.

Consistent caps

Closed by commit rPd5a72ca98ed2: Don't show "clone-name" as "Short Name" (authored by avivey, committed by avivey). · Explain WhyNov 14 2016, 10:46 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

management/

DiffusionRepositoryBasicsManagementPanel.php

3 lines

Diff 40594

View Options

src/applications/diffusion/management/DiffusionRepositoryBasicsManagementPanel.php

Don't show "clone-name" as "Short Name"ClosedPublicActions