Page MenuHomePhabricator
Differential D7846

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

Authored by epriestley on Dec 27 2013, 6:45 PM.
Tags
None
Referenced Files
F19821283: D7846.id17746.diff
Fri, Mar 6, 8:34 PM
F19771508: D7846.id17744.diff
Fri, Feb 20, 12:49 AM
F19707477: D7846.id17744.diff
Wed, Feb 11, 11:39 AM
F19633697: D7846.id17755.diff
Feb 5 2026, 7:12 AM
F19612721: D7846.id.diff
Feb 3 2026, 7:33 PM
F19610150: D7846.id17744.diff
Feb 3 2026, 3:23 PM
F19600646: D7846.id17746.diff
Feb 3 2026, 3:59 AM
F19152724: D7846.id17744.diff
Dec 11 2025, 4:45 AM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4264: Implement Repo/Project Herald rules to provide a middle ground between personal and global rules
Commits
Restricted Diffusion Commit
rPf38a565aa5b6: Use radio buttons with explanatory text to select commit rule types
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 accepted this revision.Dec 27 2013, 9:08 PM
btrahan added inline comments.
src/applications/diffusion/herald/HeraldPreCommitContentAdapter.php
41

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

epriestley added inline comments.Dec 27 2013, 9:14 PM
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".
epriestley closed this revision.Dec 27 2013, 9:16 PM

Closed by commit rPf38a565aa5b6 (authored by @epriestley).

Revision Contents
Changeset List

Diff 17755

View Options

src/applications/diffusion/herald/HeraldPreCommitContentAdapter.php

Loading...
View Options

src/applications/diffusion/herald/HeraldPreCommitRefAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldCommitAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldDifferentialRevisionAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldManiphestTaskAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldPholioMockAdapter.php

Loading...
View Options

src/applications/herald/controller/HeraldNewController.php

Loading...
Phabricator · Built by Phacility