Page MenuHomePhabricator

Use radio buttons with explanatory text to select commit rule types
ClosedPublic

Authored by epriestley on Dec 27 2013, 6:45 PM.
Tags
None
Referenced Files
F15459320: D7846.id17755.diff
Mon, Mar 31, 11:53 AM
F15454568: D7846.id17744.diff
Sat, Mar 29, 7:07 PM
F15452670: D7846.id17746.diff
Sat, Mar 29, 6:36 AM
F15450713: D7846.diff
Fri, Mar 28, 5:16 PM
F15383938: D7846.id17746.diff
Fri, Mar 14, 6:22 PM
F15368449: D7846.diff
Wed, Mar 12, 2:14 AM
F15338552: D7846.id17755.diff
Sun, Mar 9, 11:24 AM
F15333176: D7846.id.diff
Sat, Mar 8, 12:29 AM
Subscribers

Details

Summary

Ref T4264. Instead of a dropdown, make this step more informative.

Test Plan

{F93928}

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan added inline comments.
src/applications/diffusion/herald/HeraldPreCommitContentAdapter.php
41

"Hook rules can block changes." sounds a little better to me

src/applications/diffusion/herald/HeraldPreCommitContentAdapter.php
41

Oops, thanks.

epriestley updated this revision to Unknown Object (????).Dec 27 2013, 9:15 PM
  • Replace nonsense "Hooks rules" with grammatical "Hook rules".