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
F14043603: D10270.diff
Tue, Nov 12, 11:37 AM
F14025730: D10270.diff
Thu, Nov 7, 7:54 PM
F14015410: D10270.id.diff
Sun, Nov 3, 6:25 PM
F13993829: D10270.id.diff
Wed, Oct 23, 1:40 AM
F13973341: D10270.id24731.diff
Fri, Oct 18, 12:29 AM
Unknown Object (File)
Oct 3 2024, 7:08 AM
Unknown Object (File)
Sep 30 2024, 5:11 PM
Unknown Object (File)
Sep 30 2024, 11:24 AM
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