Page MenuHomePhabricator

Clean up "*.search" API method documentation pages
ClosedPublic

Authored by epriestley on Dec 13 2015, 6:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 12, 4:51 AM
Unknown Object (File)
Wed, Apr 10, 7:45 AM
Unknown Object (File)
Tue, Apr 9, 4:39 AM
Unknown Object (File)
Fri, Apr 5, 4:51 AM
Unknown Object (File)
Mon, Apr 1, 2:04 AM
Unknown Object (File)
Fri, Mar 29, 11:55 AM
Unknown Object (File)
Mar 5 2024, 2:03 AM
Unknown Object (File)
Mar 5 2024, 2:02 AM
Subscribers
None

Details

Summary

Ref T9964. Building tables in Remarkup is kind of neat-ish but ends up feeling kind of hacky, and requires weird workarounds if any of the values have | in them.

Switch to normal elements instead.

Also move the magic "ids" and "phids" to be more like real fields. I'll clean this up fully in a diff or two, it's just a little tricky because Maniphest has an "ids" field.

Test Plan

Screen Shot 2015-12-13 at 10.01.58 AM.png (641×948 px, 111 KB)

Diff Detail

Repository
rP Phabricator
Branch
nx10
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/search/engine/PhabricatorApplicationSearchEngine.php:1078XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 9525
Build 11370: Run Core Tests
Build 11369: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Clean up "*.search" API method documentation pages.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 13 2015, 7:58 PM
This revision was automatically updated to reflect the committed changes.