Page MenuHomePhabricator

Fix two minor display issues with the Conduit "*.search" API documentation
ClosedPublic

Authored by epriestley on Jul 31 2019, 6:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 9, 3:49 PM
Unknown Object (File)
Sat, Nov 9, 4:20 AM
Unknown Object (File)
Sat, Nov 9, 4:06 AM
Unknown Object (File)
Fri, Oct 25, 11:46 PM
Unknown Object (File)
Thu, Oct 24, 12:14 PM
Unknown Object (File)
Oct 19 2024, 4:12 AM
Unknown Object (File)
Oct 19 2024, 4:12 AM
Unknown Object (File)
Oct 11 2024, 12:08 PM
Subscribers
None

Details

Summary

Depends on D20685. Ref T13350. Currently:

  • When a SearchEngine parameter is marked as hidden from Conduit, we may still render a table of possible values. Instead, only render the table if the parameter is actually usable.
  • The table header is hard-coded to say 'statuses', which is just a silly mistake. (Most commonly, this table does have statuses constants.)
Test Plan

Viewed the Conduit API documentation for the new "slowvote.poll.search" API method, saw more sensible display behavior.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable