Page MenuHomePhabricator
Differential D10270

Remarkup - add a regex to blacklist what objects get link
ClosedPublic
Actions

Authored by btrahan on Aug 14 2014, 10:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 30, 10:09 PM
Unknown Object (File)
Sat, Jan 25, 10:06 PM
Unknown Object (File)
Thu, Jan 23, 3:34 PM
Unknown Object (File)
Thu, Jan 23, 3:34 PM
Unknown Object (File)
Thu, Jan 23, 3:34 PM
Unknown Object (File)
Thu, Jan 23, 3:34 PM
Unknown Object (File)
Sun, Jan 12, 11:47 PM
Unknown Object (File)
Sun, Jan 12, 4:13 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T5453: Blacklist monograms which are also used in normal discussions, like "Q1" (quarter 1)
Commits
Restricted Diffusion Commit
rPff51a1a451f1: Remarkup - add a regex to blacklist what objects get link
Summary

Fixes T5453.

Test Plan

made a remarkup comment that "Q1 is dumb and Q10 is awesome" and only Q10 was linked. changed the new setting to have the value " " and the Q1 also started linking.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan updated this revision to Diff 24730.Aug 14 2014, 10:08 PM
btrahan added a task: T5453: Blacklist monograms which are also used in normal discussions, like "Q1" (quarter 1).
btrahan retitled this revision from to Remarkup - add a regex to blacklist what objects get link.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptAug 14 2014, 10:08 PM
Harbormaster completed remote builds in B2233: Diff 24730.Aug 14 2014, 10:09 PM
epriestley accepted this revision.Aug 14 2014, 10:11 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 14 2014, 10:11 PM
btrahan closed this revision.Aug 14 2014, 10:17 PM
btrahan updated this revision to Diff 24731.

Closed by commit rPff51a1a451f1 (authored by @btrahan).

Revision Contents
Changeset List

PathSize
src/
applications/
config/
option/
12 lines
ponder/
remarkup/
12 lines
docs/
user/
userguide/
10 lines
infrastructure/
markup/
rule/
6 lines

Diff 24731

View Options

src/applications/config/option/PhabricatorCoreConfigOptions.php

Loading...
View Options

src/applications/ponder/remarkup/PonderRemarkupRule.php

Loading...
View Options

src/docs/user/userguide/remarkup.diviner

Loading...
View Options

src/infrastructure/markup/rule/PhabricatorObjectRemarkupRule.php

Loading...
Phabricator · Built by Phacility