Page MenuHomePhabricator

Give audits a clickable commit title, even if they are importing or messageless
ClosedPublic

Authored by epriestley on May 17 2014, 10:04 PM.
Tags
None
Referenced Files
F14069926: D9169.diff
Wed, Nov 20, 10:36 AM
F14064085: D9169.diff
Mon, Nov 18, 9:44 PM
F14040316: D9169.id21781.diff
Mon, Nov 11, 9:05 AM
F14033475: D9169.id21781.diff
Sat, Nov 9, 5:43 PM
F14033474: D9169.id21779.diff
Sat, Nov 9, 5:43 PM
F14033473: D9169.id.diff
Sat, Nov 9, 5:43 PM
F14030910: D9169.id21781.diff
Sat, Nov 9, 8:33 AM
F14020569: D9169.diff
Wed, Nov 6, 12:45 AM
Subscribers

Details

Reviewers
btrahan
chad
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP28c198d9026e: Give audits a clickable commit title, even if they are importing or messageless
Summary

Fixes T3854. Subversion allows commits with no message, and in other cases we might not have imported the message yet. In these cases, we may not render any text inside the link.

When we hit these cases, render appropriate replacement text.

Test Plan

Screen_Shot_2014-05-17_at_3.03.41_PM.png (67×587 px, 11 KB)

Diff Detail

Repository
rP Phabricator
Branch
atitle
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 534
Build 534: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Give audits a clickable commit title, even if they are importing or messageless.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
epriestley added a task: Restricted Maniphest Task.
chad edited edge metadata.
This revision is now accepted and ready to land.May 17 2014, 10:20 PM
epriestley updated this revision to Diff 21781.

Closed by commit rP28c198d9026e (authored by @epriestley).