Page MenuHomePhabricator

Recognize Maniphest task description as a remarkup block
ClosedPublic

Authored by epriestley on Aug 4 2014, 4:21 PM.
Tags
None
Referenced Files
F15193912: D10139.id24399.diff
Sat, Feb 22, 7:16 AM
Unknown Object (File)
Tue, Feb 18, 7:35 AM
Unknown Object (File)
Fri, Feb 7, 10:11 PM
Unknown Object (File)
Fri, Feb 7, 10:11 PM
Unknown Object (File)
Fri, Feb 7, 10:11 PM
Unknown Object (File)
Fri, Feb 7, 10:11 PM
Unknown Object (File)
Tue, Feb 4, 9:00 PM
Unknown Object (File)
Mon, Feb 3, 11:21 PM
Subscribers

Details

Summary

Ref T4589. We don't recognize task descriptions as remarkup blocks, so {F...} references in them do not get attached to the objects, and thus no policy exemption is created.

Recognize them, which activates {F...} and @mentions.

We probably have a few more of these in other applications, but it's not a big deal to clean them up as they arise.

Test Plan

Uploaded a file to a task in the description field, saw it attach and get a policy exemption.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Recognize Maniphest task description as a remarkup block.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 4 2014, 6:52 PM
epriestley updated this revision to Diff 24399.

Closed by commit rP86dbf1d17d20 (authored by @epriestley).