Page MenuHomePhabricator

Make repository callsigns optional
ClosedPublic

Authored by epriestley on Feb 18 2016, 2:20 PM.
Tags
None
Referenced Files
F14847785: D15305.diff
Mon, Feb 3, 3:46 PM
Unknown Object (File)
Tue, Jan 28, 12:58 AM
Unknown Object (File)
Fri, Jan 24, 11:53 AM
Unknown Object (File)
Fri, Jan 24, 5:33 AM
Unknown Object (File)
Tue, Jan 21, 9:44 AM
Unknown Object (File)
Sun, Jan 12, 11:14 AM
Unknown Object (File)
Dec 19 2024, 2:21 PM
Unknown Object (File)
Dec 13 2024, 11:34 AM
Subscribers
Tokens
"Mountain of Wealth" token, awarded by techdragon.

Details

Summary

Ref T4245. This could still use a little UI smoothing, but:

  • Don't require a callsign on the create flow (you can add one later in "Edit Basic Information" if you want).
  • Allow existing callsigns to be removed.
Test Plan
  • Created a new repository with no callsign.
  • Cloned it; pushed to it.
  • Browsed around Diffusion a bunch.
  • Visited a commit URI.
  • Added a callsign to it.
  • Removed the callsign again.
  • Referenced it with R22 in remarkup.

Diff Detail

Repository
rP Phabricator
Branch
call11
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10757
Build 13242: Run Core Tests
Build 13241: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 36915.
epriestley retitled this revision from to Make repository callsigns optional.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
  • Fix one issue with {abcdef} on a hash in a callsign-free repository.
chad edited edge metadata.

This could still use a little UI smoothing

imgres.jpg (225×225 px, 8 KB)

This revision is now accepted and ready to land.Feb 18 2016, 4:20 PM
This revision was automatically updated to reflect the committed changes.